-
Notifications
You must be signed in to change notification settings - Fork 3
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
Parsing for values > 999 EUR wrong #4
Comments
try this file (I cannot test it => my account is not over 1000€ ;) ): you need the test/beta version to use it... |
Unfortunately, I can't test the changes without having a signed extension. I'm running the version from the App Store and this doesn't allow unsigned extensions (not sure why the developer decided so). |
I think you can remove the app and install the download version... |
ok, finally got it working, with your patch @lr1980, using the English format, I added another gsub and removed the comma, resulting in something like this:
May I ask you to create the PR? I'm too lazy :-) |
Having more than 1.000 EUR in the account, the parsing mixes the thousand separators '.' with the decimal separator, thus the parsed number will become e.g. 9.000.00 instead of 9,000.00 or 9000.00. The application throws an error:
estateguru.lua: expected number for 'securities.purchasePrice', securities name 'Account Summary'
Unfortunately, I can't fix it myself, I have the app from the app store and it doesn't allow extension without a signature
The text was updated successfully, but these errors were encountered: