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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tax displays incorrectly on totals below $1 in outbound shipments #4280

Closed
lache-melvin opened this issue Jun 26, 2024 · 5 comments 路 Fixed by #4330
Closed

Tax displays incorrectly on totals below $1 in outbound shipments #4280

lache-melvin opened this issue Jun 26, 2024 · 5 comments 路 Fixed by #4330
Assignees
Labels
bug Something is borken Build Tested: Passed Issue has been build tested with all tests passed. Needs Investigation Not sure what's the cause of issue Severity: Normal Bugs which have an acceptable workaround. Moderate/tolerable user impact. Next minor release. Team Ruru 馃 Lach茅, Carl, Andrei Tester: Taina
Milestone

Comments

@lache-melvin
Copy link
Contributor

lache-melvin commented Jun 26, 2024

What went wrong? 馃槻

Seems fairly edge case (maybe less so for service charges?) but still a bit random...

If sub-total is less that $1, the tax percentage display gets confused - seems to show the whatever the tax value as a percentage, rather than the tax percentage?

In this case, I set tax to 50% for both the service charge and the items, which you can see the totals are calculated correctly, but the percentages are then 40% and 10%:

Screenshot 2024-06-26 at 12 02 11鈥疨M

When you open the tax edit modal, it correctly shows 50 before refreshing the page, then it shows incorrectly (e.g. 10).

Expected behaviour 馃

Tax percentage displays the tax percentage 馃憖

How to Reproduce 馃敤

Steps to reproduce the behaviour:

  1. Go to Outbound Shipment
  2. Adds some items and/or service charges where total adds to less than a dollar
  3. Add tax percentage
  4. See tax percentage displayed is not the tax percentage you entered

Your environment 馃尡

  • Open mSupply Version: 2.1 RC
  • Legacy mSupply Central Server Version:
  • Platform:
  • Database type:
@lache-melvin lache-melvin added bug Something is borken needs triage needs daily triage New bug to be triaged by the release team labels Jun 26, 2024
@lache-melvin lache-melvin changed the title Tax displays incorrectly on totals below $1 Tax displays incorrectly on totals below $1 in outbound shipments Jun 26, 2024
@andreievg andreievg added the Needs Investigation Not sure what's the cause of issue label Jun 27, 2024
@mark-prins mark-prins removed needs triage needs daily triage New bug to be triaged by the release team labels Jun 27, 2024
@andreievg
Copy link
Collaborator

Triage, if it's just a front end bug then we can get it into 2.1, but if large problem then can be carried over to 2.2

@andreievg andreievg added Team Ruru 馃 Lach茅, Carl, Andrei Severity: Normal Bugs which have an acceptable workaround. Moderate/tolerable user impact. Next minor release. labels Jun 27, 2024
@andreievg andreievg added this to the V2.1.0 milestone Jun 27, 2024
@andreievg
Copy link
Collaborator

Related: #4281

@roxy-dao
Copy link
Contributor

roxy-dao commented Jul 2, 2024

Closed in #4330

@roxy-dao roxy-dao closed this as completed Jul 2, 2024
@roxy-dao roxy-dao modified the milestones: v2.1.0-rc6, V2.1.0 Jul 3, 2024
@regotaina regotaina added Release Testing For Release Testing issues, created for testing suites Tester: Taina labels Jul 4, 2024
@regotaina
Copy link
Contributor

regotaina commented Jul 4, 2024

馃И Testing

  • Create an outbound shipment

  • Add lines with sell price less than $1

  • Add tax

  • Should calculate tax correctly

  • Have service line in master list for store

  • Add multiple service charges with different tax percentages

  • Tax percentage and total should calculate correctly

@marthakeezy marthakeezy removed the Release Testing For Release Testing issues, created for testing suites label Jul 4, 2024
@regotaina
Copy link
Contributor

Passed the test 馃

Image

@regotaina regotaina added the Build Tested: Passed Issue has been build tested with all tests passed. label Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is borken Build Tested: Passed Issue has been build tested with all tests passed. Needs Investigation Not sure what's the cause of issue Severity: Normal Bugs which have an acceptable workaround. Moderate/tolerable user impact. Next minor release. Team Ruru 馃 Lach茅, Carl, Andrei Tester: Taina
Projects
None yet
6 participants