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

[Bug]: New Inventory Currency does not Default to Config #211

Closed
JamesLPearson opened this issue Jun 23, 2023 · 2 comments · Fixed by #216
Closed

[Bug]: New Inventory Currency does not Default to Config #211

JamesLPearson opened this issue Jun 23, 2023 · 2 comments · Fixed by #216
Assignees
Labels
bug Something isn't working

Comments

@JamesLPearson
Copy link

Binner version

v2.5.9

Operating System

Windows

Describe the bug and the steps to reproduce it

When adding a new inventory item it would be good if the cost defaulted to the one in the configuration. Instead I have to change from USD to GBP every time.

Also, when adding a custom supplier there is no currency associated with the Cost field. After adding a new supplier it shows up as US$, then when you refresh the page it changes to show £ (GBP) - in my case.

Would you like to attach your appsetings.json configuration?

No response

Screenshots or Videos (Optional, but they help!)

No response

Are you able to contribute a PR? (No is ok!)

None

@JamesLPearson JamesLPearson added the bug Something isn't working label Jun 23, 2023
@replaysMike replaysMike self-assigned this Jun 27, 2023
@replaysMike
Copy link
Owner

Added in v2.6.0

replaysMike added a commit that referenced this issue Jun 27, 2023
Added support for editing currency/language to Settings page. Defaulted currency to configured value, unless provided otherwise from backend api data.
@JamesLPearson
Copy link
Author

Hi @replaysMike, doesn't look like this issue has been resolved. Still present in V2.6.0 (defaulting to US$)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants