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

Update env-template file #1073

Merged
merged 2 commits into from
Feb 25, 2021
Merged

Conversation

pnowosie
Copy link
Contributor

Follow up to #1069
Fixes PAY-206

.env-template Outdated Show resolved Hide resolved
Co-authored-by: Adam Wierzbicki <[email protected]>
@pnowosie pnowosie requested a review from Wiezzel February 24, 2021 12:10
@Wiezzel
Copy link

Wiezzel commented Feb 24, 2021

@pnowosie Sorry, I've mislead you with my suggestion. ACCOUNT_LIST env variable should be included.

@pnowosie
Copy link
Contributor Author

@pnowosie Sorry, I've mislead you with my suggestion. ACCOUNT_LIST env variable should be included.

Correct, it is still there. I've just removed the duplicate

@Wiezzel
Copy link

Wiezzel commented Feb 25, 2021

@pnowosie Ah, so I've unintentionally posted the right suggestion. That's called subconscious programming.

Copy link
Contributor

@maaktweluit maaktweluit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@pnowosie pnowosie merged commit e6c7a26 into release/v0.6 Feb 25, 2021
@pnowosie pnowosie deleted the pnowosie/pay-206-env-template-update branch February 25, 2021 10:57
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