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’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

Closed
6 tasks done
izarutskaya opened this issue Feb 28, 2024 · 20 comments
Closed
6 tasks done
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Needs Reproduction Reproducible steps needed retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause Weekly KSv2

Comments

@izarutskaya
Copy link

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com 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:

  • User is an employee of Collect workspace.
  • The Collect workspace has tax enabled.
  1. Go to workspace chat.
  2. Click + > Request money > Manual.
  3. Enter amount > Next.
  4. Click Show more.
  5. Note that tax amount is computed.
  6. Go to FAB > Request money > Manual.
  7. Enter amount.
  8. Select the Collect workspace.
  9. Click Show more.

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?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Bug6395218_1709110829868.20240228_131855.mp4

View all open jobs on GitHub

@izarutskaya izarutskaya added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Feb 28, 2024
Copy link

melvin-bot bot commented Feb 28, 2024

Triggered auto assignment to @strepanier03 (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@izarutskaya
Copy link
Author

@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.

@izarutskaya
Copy link
Author

We think that this bug might be related to #wave6-collect-submitters
CC @greg-schroeder

@MonilBhavsar
Copy link
Contributor

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

@melvin-bot melvin-bot bot added the Overdue label Mar 1, 2024
@strepanier03
Copy link
Contributor

Sounds good, thanks @MonilBhavsar!

@melvin-bot melvin-bot bot removed the Overdue label Mar 1, 2024
@strepanier03 strepanier03 changed the title Tax amount is not automatically computed when creating request via FAB [HOLD #305057) Tax amount is not automatically computed when creating request via FAB Mar 1, 2024
@strepanier03 strepanier03 added Weekly KSv2 and removed Daily KSv2 labels Mar 1, 2024
@melvin-bot melvin-bot bot added the Overdue label Mar 11, 2024
@strepanier03
Copy link
Contributor

Hold still.

@melvin-bot melvin-bot bot removed the Overdue label Mar 13, 2024
@melvin-bot melvin-bot bot added the Overdue label Mar 25, 2024
@strepanier03
Copy link
Contributor

Still on Hold.

@melvin-bot melvin-bot bot removed the Overdue label Mar 27, 2024
@melvin-bot melvin-bot bot added the Overdue label Apr 5, 2024
@strepanier03
Copy link
Contributor

Still paused while HOLD GH is open.

@melvin-bot melvin-bot bot removed the Overdue label Apr 16, 2024
@MonilBhavsar
Copy link
Contributor

@strepanier03 this issue is now fixed. Feel free to retest and close it

@strepanier03
Copy link
Contributor

Great, I'll test and update. Thanks @MonilBhavsar!

@strepanier03
Copy link
Contributor

Hmmm, this still isn't working for me on Safari at least, testing on Chrome as well, then I'll update with video/screenshots.

@strepanier03
Copy link
Contributor

Testing notes

Collect Workspace tax setup
image


No 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.mp4

The behavior is the same in Chrome as well.

2024-04-17_14-53-32.mp4

@strepanier03
Copy link
Contributor

@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.

@strepanier03 strepanier03 changed the title [HOLD #305057) Tax amount is not automatically computed when creating request via FAB [Retesting) Tax amount is not automatically computed when creating request via FAB Apr 17, 2024
@melvin-bot melvin-bot bot added the Overdue label Apr 26, 2024
@strepanier03 strepanier03 added the retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause label May 3, 2024
@strepanier03 strepanier03 removed the retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause label May 3, 2024
@strepanier03
Copy link
Contributor

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.

FAB:
image

From Workspace chat > small + > Submit expense
image

@MonilBhavsar - Am I holding this wrong somehow? Are you able to get an expense's tax amount to calculate automatically without toggling?

@melvin-bot melvin-bot bot removed the Overdue label May 3, 2024
@melvin-bot melvin-bot bot added the Overdue label May 13, 2024
@strepanier03
Copy link
Contributor

@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.

@melvin-bot melvin-bot bot removed the Overdue label May 14, 2024
@MonilBhavsar
Copy link
Contributor

Interesting, I'll retest today

@MonilBhavsar
Copy link
Contributor

MonilBhavsar commented May 15, 2024

Tax amount is showing correct for me

Screen.Recording.2024-05-15.at.11.05.48.AM.mov

@MonilBhavsar MonilBhavsar self-assigned this May 15, 2024
@MonilBhavsar MonilBhavsar added Needs Reproduction Reproducible steps needed retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause labels May 15, 2024
@MonilBhavsar MonilBhavsar changed the title [Retesting) Tax amount is not automatically computed when creating request via FAB Tax amount is not automatically computed when creating request via FAB May 15, 2024
@MelvinBot
Copy link

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

@mvtglobally
Copy link

Issue not reproducible during KI retests. (First week)

@MonilBhavsar
Copy link
Contributor

Thanks! Let's close it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Needs Reproduction Reproducible steps needed retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause Weekly KSv2
Projects
Archived in project
Development

No branches or pull requests

5 participants