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

[$500] New order of tags do not update after reopening confirmation page #37864

Closed
6 tasks done
m-natarajan opened this issue Mar 7, 2024 · 19 comments
Closed
6 tasks done
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering Internal Requires API changes or must be handled by Expensify staff retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause

Comments

@m-natarajan
Copy link

m-natarajan commented Mar 7, 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.48-0
Reproducible in staging?: y
Reproducible in production?: New Feature
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers):
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 independent multilevel tags.
  1. [Employee] Go to workspace chat.
  2. {Employee] Go to FAB > Request money > Manual > Enter amount.
  3. [Employee] On confirmation page, note the order of multilevel tags.
  4. [Admin] On Old Dot, rearrange the order of multilevel tags (do not delete them)
  5. [Employee] Without refreshing the page, repeat Step 2 and 3.

Expected Result:

The new order of multilevel tags will update when reopening the confirmation page, as it works (it updates) when changing tag name.

Actual Result:

The new order of multilevel tags does not update when reopening the confirmation page. It only updates after refreshing the page.

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

Bug6404768_1709772124813.bandicam_2024-03-07_08-32-26-892.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~0194dd2ee1e3003f2e
  • Upwork Job ID: 1768169703946260480
  • Last Price Increase: 2024-03-14
@m-natarajan m-natarajan added DeployBlockerCash This issue or pull request should block deployment Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Mar 7, 2024
Copy link

melvin-bot bot commented Mar 7, 2024

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

@github-actions github-actions bot added Engineering Hourly KSv2 and removed Daily KSv2 labels Mar 7, 2024
Copy link
Contributor

github-actions bot commented Mar 7, 2024

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.

Copy link

melvin-bot bot commented Mar 7, 2024

Triggered auto assignment to @robertjchen (Engineering), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

@m-natarajan
Copy link
Author

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

@m-natarajan
Copy link
Author

We think that this bug might be related to #wave6-collect-submitters

cc@greg-schroeder

@robertjchen robertjchen removed the DeployBlockerCash This issue or pull request should block deployment label Mar 7, 2024
@robertjchen
Copy link
Contributor

Yep, not a deploy blocker per se, but definitely looks like a bug to me. Though, I'm not sure if it's on the backend cc: @greg-schroeder

@robertjchen robertjchen added Daily KSv2 and removed Hourly KSv2 labels Mar 7, 2024
@luacmartins
Copy link
Contributor

Seems like we're not sending pusher updates for the reordering. Maybe something that https://github.com/Expensify/Auth/pull/9773 will solve

@robertjchen robertjchen added Weekly KSv2 and removed Daily KSv2 labels Mar 8, 2024
@robertjchen
Copy link
Contributor

That makes sense, let's revisit after that change goes out.

@luacmartins luacmartins changed the title New order of tags do not update after reopening confirmation page [HOLD Auth #9773] New order of tags do not update after reopening confirmation page Mar 11, 2024
@luacmartins
Copy link
Contributor

Cool, put a hold for that Auth PR - https://github.com/Expensify/Auth/pull/9773

@Christinadobrzyn
Copy link
Contributor

PR is closed https://github.com/Expensify/Auth/pull/9773

Testing the steps in the OP again, I think this might be fixed!

2024-03-13_12-12-16.mp4

Gonna ask QA to test - https://expensify.slack.com/archives/C9YU7BX5M/p1710303354198479

@kavimuru
Copy link

@Christinadobrzyn Bug is still reproducible by the tester.

bandicam.2024-03-13.20-49-38-989.mp4

@Christinadobrzyn
Copy link
Contributor

Christinadobrzyn commented Mar 14, 2024

Ah okay, I deleted the tags in my test here, not moved them so that is the difference. I can reproduce the steps in the OP.

I think this might be internal, @luacmartins @robertjchen should this remain be internal?

@Christinadobrzyn Christinadobrzyn changed the title [HOLD Auth #9773] New order of tags do not update after reopening confirmation page [$500] New order of tags do not update after reopening confirmation page Mar 14, 2024
Copy link

melvin-bot bot commented Mar 14, 2024

⚠️ Could not update price automatically because there is no linked Upwork Job ID. The BZ team member will need to update the price manually in Upwork.

@Christinadobrzyn Christinadobrzyn added the Internal Requires API changes or must be handled by Expensify staff label Mar 14, 2024
Copy link

melvin-bot bot commented Mar 14, 2024

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

Copy link

melvin-bot bot commented Mar 14, 2024

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

@robertjchen
Copy link
Contributor

The fix was deployed last week. @m-natarajan could you please help confirm if the issue is still reproducible? 🙏

@robertjchen robertjchen added the retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause label Mar 21, 2024
@Christinadobrzyn
Copy link
Contributor

Asking QA to test too - I think this is resolved - https://expensify.slack.com/archives/C9YU7BX5M/p1711355439541929

@Christinadobrzyn Christinadobrzyn added Daily KSv2 and removed Weekly KSv2 labels Mar 25, 2024
@kavimuru
Copy link

@Christinadobrzyn issue not reproducible anymore.

20240325_222015.mp4

@robertjchen
Copy link
Contributor

Thanks for helping test, closing this out 👍

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 Engineering Internal Requires API changes or must be handled by Expensify staff retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause
Projects
Archived in project
Development

No branches or pull requests

6 participants