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

Issue with currency decimal digits #53

Closed
redispade opened this issue Feb 12, 2019 · 5 comments
Closed

Issue with currency decimal digits #53

redispade opened this issue Feb 12, 2019 · 5 comments
Labels
enhancement New feature or request

Comments

@redispade
Copy link

Hello.
I created a custom currency in the database(Albanian Lek, didn't exist on the list). I assigned zero decimal digits and in the main program works correctly. However there is an issue in the mobile app, it adds two decimal digits and messes up with the values. For example I have an invoice where I have value of 1667, VAT(20%) of 333 and total of 2000. Instead on the mobile app I have value of 1666, VAT of 333.4 and a total of 1999.4 Since I have set zero decimal digits, the web app is OK, but the mobile app does not respect that.

@hillelcoren
Copy link
Member

The app doesn't support zero digit precision, there will likely be other problems with the web app.

@redispade
Copy link
Author

What kind of problems ? The smallest unit of currency in our country is 1 Lek. I usually round Vat to that(in excess) even when issuing paper invoices.

@hillelcoren
Copy link
Member

We hope to support this with v2 of the app.

Sorry, I don't remember exactly but in the past we had to change other zero digit currencies to two digits.

@redispade
Copy link
Author

If the problem is on the mobile app only, it's ok for now, it's a minor issue, as long as the server side is good.
I am using it self hosted and seriously considering whitelabeling. If you need to test anything feel free to drop a message

@hillelcoren hillelcoren added the enhancement New feature or request label Feb 12, 2019
@hillelcoren
Copy link
Member

This has been implemented in the v5 app

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants