search

abidlabs / zakat.fyi

A well-designed modern zakat web app
https://zakat.fyi
MIT License
9 stars 1 forks source link

Passive Investments Bug #113

Closed itinawi closed 4 years ago

itinawi commented 4 years ago

"if you have a comma in any of your numbers it throws the math off when doing Passive Investments"

abidlabs commented 4 years ago

I think this is more generally true in any field, right?

itinawi commented 4 years ago

For other fields, one comma seems to work. Multiple comma breaks it.

For passive investment, it seems that anything after the comma isn't taken into account.

abidlabs commented 4 years ago

Fixed. Also added comma to display amounts