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

[HOLD] - [$500] Violation - Expense violation message is missing from Employee's view and no amount #34807

Closed
6 tasks done
lanitochka17 opened this issue Jan 19, 2024 · 14 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. External Added to denote the issue can be worked on by a contributor Weekly KSv2

Comments

@lanitochka17
Copy link

lanitochka17 commented Jan 19, 2024

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Version Number: 1.4.28-0
Reproducible in staging?: Y
Reproducible in production?: Y
Issue reported by: Applause - Internal Team

Issue found when executing PR #31448

Action Performed:

Precondition:

  • Admin and employee are part of Control workspace in Old Dot
  • The Control workspace has set up Max Expense Amount violation
  1. [Employee] Go to workspace chat
  2. [Employee] Create a manual request that exceeds the maximum amount
  3. [Employee] Submit the request
  4. [Employee] Go to request details page
  5. [Employee] Note that there is no "Expense exceeds auto approval limit of" message
  6. [Admin] Go to the request details page

Expected Result:

In Step 5, "Expense exceeds auto approval limit of" message will be present from Employee's view
In Step 6, "Expense exceeds auto approval limit of" message will be complete with the maximum amount allowed for the violation

Actual Result:

✅ In Step 5, "Expense exceeds auto approval limit of" message is not present from Employee's view
In Step 6, "Expense exceeds auto approval limit of" message is missing maximum amount allowed for the violation

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

Add any screenshot/video evidence

Bug6347711_1705676185324.20240119_202533.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~019aaa92dcff420820
  • Upwork Job ID: 1748367750109818880
  • Last Price Increase: 2024-01-19
@lanitochka17 lanitochka17 added External Added to denote the issue can be worked on by a contributor Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Jan 19, 2024
@melvin-bot melvin-bot bot changed the title Violation - Expense violation message is missing from Employee's view and no amount [$500] Violation - Expense violation message is missing from Employee's view and no amount Jan 19, 2024
Copy link

melvin-bot bot commented Jan 19, 2024

Job added to Upwork: https://www.upwork.com/jobs/~019aaa92dcff420820

Copy link

melvin-bot bot commented Jan 19, 2024

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

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Jan 19, 2024
Copy link

melvin-bot bot commented Jan 19, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @situchan (External)

@lanitochka17
Copy link
Author

lanitochka17 commented Jan 19, 2024

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

@cead22 cead22 removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Jan 19, 2024
@cead22 cead22 assigned cead22 and unassigned situchan and jliexpensify Jan 19, 2024
@cead22
Copy link
Contributor

cead22 commented Jan 19, 2024

I'm taking this one, to check if the fix is on the backend first.

In Step 5, "Expense exceeds auto approval limit of" message will be present from Employee's view

For context: we don't show this to the person requesting money because they can't do anything about this violation

@cead22
Copy link
Contributor

cead22 commented Jan 24, 2024

Planning on triaging this today

@melvin-bot melvin-bot bot removed the Overdue label Jan 24, 2024
@cead22
Copy link
Contributor

cead22 commented Jan 24, 2024

After testing this while on this branch https://github.com/Expensify/App/pull/34303/files, I think this issue will be fixed by that PR, so putting it on HOLD for now

@cead22 cead22 changed the title [$500] Violation - Expense violation message is missing from Employee's view and no amount [HOLD] - [$500] Violation - Expense violation message is missing from Employee's view and no amount Jan 24, 2024
@melvin-bot melvin-bot bot added the Overdue label Jan 29, 2024
Copy link

melvin-bot bot commented Jan 30, 2024

@cead22 Eep! 4 days overdue now. Issues have feelings too...

@cead22
Copy link
Contributor

cead22 commented Jan 31, 2024

On hold, not overdue

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Jan 31, 2024
Copy link

melvin-bot bot commented Feb 5, 2024

@cead22 Huh... This is 4 days overdue. Who can take care of this?

Copy link

melvin-bot bot commented Feb 7, 2024

@cead22 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

@cead22 cead22 added the Weekly KSv2 label Feb 8, 2024
@cead22 cead22 removed the Daily KSv2 label Feb 8, 2024
@melvin-bot melvin-bot bot removed the Overdue label Feb 8, 2024
@JmillsExpensify
Copy link

Linked issue still hasn't merged, though it looks very close.

@cead22
Copy link
Contributor

cead22 commented Feb 13, 2024

PR got merged but hasn't been deployed yet

@melvin-bot melvin-bot bot added the Overdue label Feb 21, 2024
@cead22
Copy link
Contributor

cead22 commented Feb 21, 2024

This should be fixed now

@cead22 cead22 closed this as completed Feb 21, 2024
@melvin-bot melvin-bot bot removed the Overdue label Feb 21, 2024
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. External Added to denote the issue can be worked on by a contributor Weekly KSv2
Projects
None yet
Development

No branches or pull requests

5 participants