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

fix(config): MAX_PRICE quotations #426

Merged
merged 4 commits into from Oct 3, 2020
Merged

fix(config): MAX_PRICE quotations #426

merged 4 commits into from Oct 3, 2020

Conversation

xninjax
Copy link
Contributor

@xninjax xninjax commented Oct 2, 2020

bug fix: Incorrect quotations used in .env-example for MAX_PRICE

Description

Fixes #428

bug fixing an oops 😨... previous merge of #383 had contained an error within .env-example.
Abnormal quotations somehow snuck in with the MAX_PRICE default.

MAX_PRICE=β€œβ€

       these ^^^ are curly quotations.  Should be straight quotations.

Testing

Testing with a clean clone of the newly merged commit caused price checking to fail.
Discovered the abnormal quotation marks within .env-example
Updated quotations for MAX_PRICE="" and now working again.

πŸ€·β€β™‚οΈ

@xninjax
Copy link
Contributor Author

xninjax commented Oct 2, 2020

@jef When my discord alerts started pinging hard I thought I was about to get a new card!.. then realizing it was amazon scalper cards well over my price limit, I knew something was wrong 😠

Honestly I don't even know how to produce those curly quotes directly, so not sure how those sneaky little chars really made their way into .env-example...

anyhow, easy fix, but feel responsible for that getting out into the wild πŸ€·β€β™‚οΈ

btw congrats on your 3080 πŸ† πŸ˜„

@jef
Copy link
Owner

jef commented Oct 3, 2020

Thanks for being so thorough on your description and debugging. I appreciate it ❀️

@jef jef changed the title Xninjax/price check bugfix fix(config): MAX_PRICE quotations Oct 3, 2020
@jef jef merged commit b7e5941 into jef:main Oct 3, 2020
@xninjax xninjax deleted the xninjax/price-check-bugfix branch October 3, 2020 17:30
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.

psa: MAX_PRICE default in .env-example uses incorrect quotation marks
2 participants