App icon indicating copy to clipboard operation
App copied to clipboard

[$250] Report action says automatically approved $0.00 via

Open m-natarajan opened this issue 1 year ago β€’ 17 comments

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: Reproducible in staging?: need reproduction Reproducible in production?: need reproduction If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: 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: @flodnv Slack conversation (hyperlinked to channel name): expensify-migrate

Action Performed:

  1. Have a report submitted and approved on a OD
  2. Open the report in ND

Expected Result:

Should show correct amount of the approved expense

Actual Result:

Report action says automatically approved $0.00 via

Workaround:

unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • [ ] Android: Standalone
  • [ ] Android: HybridApp
  • [ ] Android: mWeb Chrome
  • [ ] iOS: Standalone
  • [ ] iOS: HybridApp
  • [ ] iOS: mWeb Safari
  • [x] MacOS: Chrome / Safari
  • [ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

image (16)

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021861506983862256537
  • Upwork Job ID: 1861506983862256537
  • Last Price Increase: 2024-12-03
Issue OwnerCurrent Issue Owner: @c3024

m-natarajan avatar Nov 23 '24 20:11 m-natarajan

Triggered auto assignment to @OfstadC (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.

melvin-bot[bot] avatar Nov 23 '24 20:11 melvin-bot[bot]

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

MelvinBot avatar Nov 23 '24 20:11 MelvinBot

@m-natarajan can you please be a bit more descriptive than OD and ND coz most of us new contributors don't know what that is?

me-ZaidAli avatar Nov 24 '24 11:11 me-ZaidAli

https://github.com/Expensify/App/blob/f4f8da74c100f2c5ddb666ff4314038010104cbf/contributingGuides/CONTRIBUTING.md?plain=1#L210-L211

missbond2017 avatar Nov 25 '24 04:11 missbond2017

Yeah definitely something weird going on with reports from OD showing on New

image image

OfstadC avatar Nov 26 '24 20:11 OfstadC

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

melvin-bot[bot] avatar Nov 26 '24 20:11 melvin-bot[bot]

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

melvin-bot[bot] avatar Nov 26 '24 20:11 melvin-bot[bot]

@OfstadC Is there any specific way to reproduce it? when I submit the report from ND it properly shows the amount image but when I try to submit from OD it never auto approves even if I create an expense with lesser amount than the Approve manually above some amount setting of the workspace. This part of the step is unclear

  1. Have a report submitted and approved on a OD

How do we submit in OD so that it will be auto approved?

FitseTLT avatar Nov 28 '24 15:11 FitseTLT

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

melvin-bot[bot] avatar Dec 02 '24 09:12 melvin-bot[bot]

How do we submit in OD so that it will be auto approved?

The workspace will need to have this set in Classic - Settings > Workspace > Members

https://help.expensify.com/articles/expensify-classic/reports/Require-review-for-over-limit-expenses

OfstadC avatar Dec 02 '24 16:12 OfstadC

@OfstadC This one ? image

I did set but when I submit the expense below that amount from ND it auto approves and the report action shows the amount properly and whenever I submit from OD it never auto-approves it requires a manual approval from the admin. It would be helpful if someone who can reproduce it record screenshots :+1:

FitseTLT avatar Dec 02 '24 21:12 FitseTLT

πŸ“£ It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? πŸ’Έ

melvin-bot[bot] avatar Dec 03 '24 16:12 melvin-bot[bot]

@OfstadC, @c3024 Still overdue 6 days?! Let's take care of this!

melvin-bot[bot] avatar Dec 04 '24 09:12 melvin-bot[bot]

whenever I submit from OD it never auto-approves it requires a manual approval from the admin.

Is the report you are testing under $200? I would set the randomly route reports to 0% as well.

OfstadC avatar Dec 04 '24 15:12 OfstadC

whenever I submit from OD it never auto-approves it requires a manual approval from the admin.

Is the report you are testing under $200? I would set the randomly route reports to 0% as well.

Yes it is under that value and also I have tried to set it to 0% @OfstadC I think it would be better to have snapshots if you can reproduce it. Thx

FitseTLT avatar Dec 04 '24 15:12 FitseTLT

Waiting for proposals!

c3024 avatar Dec 05 '24 16:12 c3024

@OfstadC @c3024 this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

melvin-bot[bot] avatar Dec 07 '24 10:12 melvin-bot[bot]

@OfstadC, @c3024 Whoops! This issue is 2 days overdue. Let's get this updated quick!

melvin-bot[bot] avatar Dec 09 '24 09:12 melvin-bot[bot]

@OfstadC I think it would be better to have snapshots if you can reproduce it. Thx

I'll see what I can do today πŸ˜ƒ

OfstadC avatar Dec 09 '24 16:12 OfstadC

Not overdue!

c3024 avatar Dec 10 '24 05:12 c3024

πŸ“£ It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? πŸ’Έ

melvin-bot[bot] avatar Dec 10 '24 16:12 melvin-bot[bot]

@OfstadC, @c3024 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

melvin-bot[bot] avatar Dec 13 '24 09:12 melvin-bot[bot]

Sorry @FitseTLT I haven't had a chance to get reproducible steps put together quite yet. I'll tackle this on Tuesday when i'm back in office. Hopefully sooner 🀞

OfstadC avatar Dec 13 '24 17:12 OfstadC

Issue not reproducible during KI retests. (First week)

mvtglobally avatar Dec 14 '24 05:12 mvtglobally

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

melvin-bot[bot] avatar Dec 17 '24 09:12 melvin-bot[bot]

πŸ“£ It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? πŸ’Έ

melvin-bot[bot] avatar Dec 17 '24 16:12 melvin-bot[bot]

Hmmm... i'm not sure how to reproduce πŸ€”

All expenses/reports from Classic should still appear in NewDot, yeah?

I created an expense report in a test account and it was auto approved, but I don't see it in New Expensify πŸ€”

OfstadC avatar Dec 17 '24 18:12 OfstadC

@OfstadC, @c3024 8 days overdue is a lot. Should this be a Weekly issue? If so, feel free to change it!

melvin-bot[bot] avatar Dec 19 '24 09:12 melvin-bot[bot]

I still can't get new reports to reproduce. But all the already migrated reports are showing this πŸ€”

OfstadC avatar Dec 19 '24 18:12 OfstadC

Not overdue!

c3024 avatar Dec 20 '24 10:12 c3024