-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[$500] Currency - Dot (.) cannot be added into amount field if MRO currency selected #30136
Comments
Job added to Upwork: https://www.upwork.com/jobs/~013d5909ed29d6d625 |
Triggered auto assignment to @dylanexpensify ( |
Bug0 Triage Checklist (Main S/O)
|
Triggered auto assignment to Contributor-plus team member for initial proposal review - @burczu ( |
Seems like this currency does not have decimals allowed.
|
I do also confirm and agree with @abdel-h66, based on the current config, the currency does not accept decimals which is the reason of not being able to use (.). => View currencyList.json fix.mov |
📣 @peymancyb! 📣
|
Contributor details |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
This is what I got The Mauritanian Ouguiya (MRO) is the official currency of Mauritania. In the context of currency, you typically cannot have less than 1 unit of the currency in a monetary transaction. The Ouguiya is subdivided into smaller units, with the most common denominations being the ouguiya and the khoums. One ouguiya is equivalent to 5 khoums. So, for practical purposes, in monetary transactions, you would typically deal with whole numbers of ouguiyas or khoums, and amounts are not expressed as fractions of an ouguiya. For example, you might have 10 ouguiyas or 50 khoums, but you wouldn't typically have something like "0.5 MRO" in a standard transaction. |
@dylanexpensify I've just checked what @shubham1206agra wrote in their comment and it seems to be kinda correct - in Mauritania there is only one coin that is less than 1 Ouguiya called Khoum which is 1/5 of Ouguiya. That's why, I guess, we have decimals disabled in the App for MRO currency, because it would hard to deal with it. So I think this may not be a bug, actually. |
@burczu, @dylanexpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
agree! thank you! |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: 1.3.88-3
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Expected Result:
Dot (.) should be entered and seen into amount field
Actual Result:
Dot (.) cannot be added into amount field if MRO currency selected
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari
Bug6245699_1697910316055.Recording__1291.mp4
MacOS: Desktop
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: