Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add environment variable overrides for key strings #6

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dgw
Copy link

@dgw dgw commented Mar 31, 2024

I'm opening this PR as a draft with only the domain-name functionality implemented so far, because the other components of #5 are basically a copy-paste job unless there's some higher-level feedback to be had about the approach I'm taking.

As previously mentioned, I've never worked in Rust before, so it seemed best to start with the smallest possible diff and pause in case I did something noob-y that should be corrected before building on this. If there's a better way to do it, I'll gladly experiment more—then build the rest of the items in the same style and push a cleaned-up history minus the WIP commits.

dgw added 2 commits March 27, 2024 23:46
Doesn't reuse code at all; the same `match` is peppered through every
place the domain name appears.

Contains unused fn that encapsulates the `match` but I haven't been able
to figure out how to make it callable from within a Maud template yet.
Actually reuses code now; I was stupid for using `async fn`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant