search

msupply-foundation / open-msupply

Open mSupply represents our most recent advancement in the Logistics Management Information System (LMIS), expanding on more than two decades of development inherited from the well-established legacy of the original mSupply.
https://msupply.foundation/open-msupply/
Other
23 stars 15 forks source link

Decimal limit breaks for recurring decimals #5296

Open roxy-dao opened 2 weeks ago

roxy-dao commented 2 weeks ago

What went wrong? 😲

Setting decimal limit to 2 breaks for recurring decimals

https://github.com/user-attachments/assets/1fc03c1b-d2d2-4e16-be11-61f720addb5d

Expected behaviour 🤔

How to Reproduce 🔨

Steps to reproduce the behaviour:

  1. Probably best test in service tax?
  2. Set different service taxes (keeping in mind that total should be a recurring number e.g. .013333, 0166666)
  3. See that it doesn't default to 2dp

Your environment 🌱

jmbrunskill commented 1 week ago

Sending for normal triage, it sounds like it's just a display issue so not too critical?