-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Tax amount is not automatically computed when creating request via FAB #37415
Comments
Triggered auto assignment to @strepanier03 ( |
@strepanier03 I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors. |
We think that this bug might be related to #wave6-collect-submitters |
Feature is WIP, so we can put this HOLD until we finish the implementation as it is being rolled out in phases https://github.com/Expensify/Expensify/issues/305057 |
Sounds good, thanks @MonilBhavsar! |
Hold still. |
Still on Hold. |
Still paused while HOLD GH is open. |
@strepanier03 this issue is now fixed. Feel free to retest and close it |
Great, I'll test and update. Thanks @MonilBhavsar! |
Hmmm, this still isn't working for me on Safari at least, testing on Chrome as well, then I'll update with video/screenshots. |
Testing notesNo matter how I create an expense, the tax amount isn't calculated, I have to toggle the rate in the tax selection dropdown to get it to calculate. Video below in Safari. 2024-04-17_14-41-25.mp4The behavior is the same in Chrome as well. 2024-04-17_14-53-32.mp4 |
@MonilBhavsar—Maybe I'm confused. Should the expected behavior be that the tax is automatically calculated? That is what I was expecting, but not what I'm experiencing. |
I'm still unable to have a tax calculated automatically. It's only calculated when I toggle the rate applied. This is unexpected from what I'm gathering in https://github.com/Expensify/Expensify/issues/305057. I made a brand new collect workspace, and tried to create an expense from both the FAB and the smaller + in the workspace thread. From Workspace chat > small + > Submit expense @MonilBhavsar - Am I holding this wrong somehow? Are you able to get an expense's tax amount to calculate automatically without toggling? |
@MonilBhavsar - I'm going to set this to External and get it looked at unless you think otherwise. I can't create an expense and have tax computed unless I toggle between two rates. |
Interesting, I'll retest today |
Tax amount is showing correct for me Screen.Recording.2024-05-15.at.11.05.48.AM.mov |
This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989 |
Issue not reproducible during KI retests. (First week) |
Thanks! Let's close it |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Found when executing PR : #36784
Version Number: v1.4.44-7
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause-Internal Team
Slack conversation:
Action Performed:
Precondition:
Expected Result:
Tax amount will be not computed.
Actual Result:
Tax amount is not computed.
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Bug6395218_1709110829868.20240228_131855.mp4
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: