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

[Help Wanted] Non-admin approvers can change the report workspace for Approved reports on OldDot #41147

Open
1 of 6 tasks
m-natarajan opened this issue Apr 27, 2024 · 8 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Help Wanted Apply this label when an issue is open to proposals by contributors Internal Requires API changes or must be handled by Expensify staff Overdue

Comments

@m-natarajan
Copy link

m-natarajan commented Apr 27, 2024

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Version Number: 1.4.67-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @garrettmknight
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1714132204273809

Action Performed:

Pre-req: Setup Collect or Control workspace with Submit & Approve - set the approver as an employee, not an admin.
Action Performed:
Employee A submits a report to Employee B
Employee B approves report
Employee B clicks the details on the Report in OldDot

Expected Result:

  • Report workspace should show the actual workspace it’s on
  • Report workspace shouldn’t be editable if Employee B isn’t an admin

Actual Result:

  • Report workspace shows a list of workspaces Employee B is an admin on, not the workspace the report is on.
  • Employee B can change the workspace even though they aren’t an admin on the existing workspace

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

Recording.3044.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01b52ee6aaf74a66ca
  • Upwork Job ID: 1784915027341197312
  • Last Price Increase: 2024-04-29
@m-natarajan m-natarajan added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Apr 27, 2024
Copy link

melvin-bot bot commented Apr 27, 2024

Triggered auto assignment to @strepanier03 (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@garrettmknight garrettmknight added the Internal Requires API changes or must be handled by Expensify staff label Apr 29, 2024
Copy link

melvin-bot bot commented Apr 29, 2024

Job added to Upwork: https://www.upwork.com/jobs/~01b52ee6aaf74a66ca

Copy link

melvin-bot bot commented Apr 29, 2024

Triggered auto assignment to Contributor Plus for review of internal employee PR - @s77rt (Internal)

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

Internal and waiting to be picked up.

@melvin-bot melvin-bot bot removed the Overdue label May 2, 2024
@strepanier03 strepanier03 added the Help Wanted Apply this label when an issue is open to proposals by contributors label May 2, 2024
@strepanier03 strepanier03 changed the title Non-admin approvers can change the report workspace for Approved reports on OldDot [Help Wanted] Non-admin approvers can change the report workspace for Approved reports on OldDot May 2, 2024
@melvin-bot melvin-bot bot added the Overdue label May 6, 2024
Copy link

melvin-bot bot commented May 7, 2024

@strepanier03, @s77rt Eep! 4 days overdue now. Issues have feelings too...

@strepanier03
Copy link
Contributor

Internal still and waiting to be picked up.

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels May 7, 2024
Copy link

melvin-bot bot commented May 11, 2024

@strepanier03 @s77rt this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

Copy link

melvin-bot bot commented May 13, 2024

@strepanier03, @s77rt Huh... This is 4 days overdue. Who can take care of this?

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. Daily KSv2 Help Wanted Apply this label when an issue is open to proposals by contributors Internal Requires API changes or must be handled by Expensify staff Overdue
Projects
Status: Release 1: Spring 2024 (May)
Development

No branches or pull requests

4 participants