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 support for loading / saving .env #40

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Add support for loading / saving .env #40

wants to merge 1 commit into from

Conversation

TheUltDev
Copy link
Contributor

@TheUltDev TheUltDev commented Dec 26, 2023

@FossPrime when you get time would it be possible to add prompting and loading of a config file as suggested by @rhildred.

These are his suggested changes, here is the relevant code from his git fork:

https://github.com/search?q=repo%3Adiy-pwa%2Fgit-pwa%20.env&type=code

I'm also open to the Secrets API.

@TheUltDev TheUltDev requested a review from FossPrime December 26, 2023 09:59
@TheUltDev TheUltDev removed the request for review from FossPrime December 26, 2023 09:59
@TheUltDev TheUltDev marked this pull request as draft December 26, 2023 09:59
@rhildred
Copy link
Contributor

I saw an email about this that suggested, "@rhildred This caused issues in deployment. Reverting. Will investigate at a later time or you can provide a PR." I am willing to investigate but it deploys fine for me.

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.

3 participants