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

Set a default CURRENCY #131

Closed

Conversation

kurtmckee
Copy link
Contributor

This fixes a NameError that occurs on platforms that don't have any of the listed locale values present. (For example, LANG=en_US.UTF-8 results in a NameError.)

Fixes #130

This fixes a `NameError` that occurs on platforms
that don't have any of the listed locale values present.
(For example, `LANG=en_US.UTF-8` results in a `NameError`.)

Fixes tlocke#130
@tlocke
Copy link
Owner

tlocke commented Apr 29, 2023

Thanks for this, I've gone with the idea of getting the currency symbol using the Python locale package, and having the default of '$' when the locale is 'C'.

@tlocke tlocke closed this Apr 29, 2023
@kurtmckee
Copy link
Contributor Author

Awesome, thanks!

@kurtmckee kurtmckee deleted the set-default-currency-issue-130 branch April 30, 2023 18:43
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.

Unable to run tests locally
2 participants