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 for payment 2024-03-29] [QAB] Optimistically update the QAB NVP when taking actions #38048

Closed
Gonals opened this issue Mar 11, 2024 · 13 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Engineering Internal Requires API changes or must be handled by Expensify staff NewFeature Something to build that is a new item. Weekly KSv2

Comments

@Gonals
Copy link
Contributor

Gonals commented Mar 11, 2024

HELD on https://github.com/Expensify/Expensify/issues/377599

When taking one of the QAB-assignable actions, we want to optimistically set the QAB on the frontend, as described in this section of the design doc.

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~010994abf23e96432d
  • Upwork Job ID: 1767200034860355584
  • Last Price Increase: 2024-03-11
@Gonals Gonals added Engineering Weekly KSv2 Internal Requires API changes or must be handled by Expensify staff NewFeature Something to build that is a new item. labels Mar 11, 2024
Copy link

melvin-bot bot commented Mar 11, 2024

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

Copy link

melvin-bot bot commented Mar 11, 2024

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

Copy link

melvin-bot bot commented Mar 11, 2024

Copy link

melvin-bot bot commented Mar 11, 2024

⚠️ It looks like this issue is labelled as a New Feature but not tied to any GitHub Project. Keep in mind that all new features should be tied to GitHub Projects in order to properly track external CAP software time ⚠️

@trjExpensify trjExpensify moved this to Release 1: Spring 2024 (May) in [#whatsnext] #wave-collect Mar 12, 2024
@Gonals Gonals self-assigned this Mar 14, 2024
@Gonals Gonals changed the title [QAB][HOLD] Optimistically update the QAB NVP when taking actions [QAB] Optimistically update the QAB NVP when taking actions Mar 14, 2024
@Gonals
Copy link
Contributor Author

Gonals commented Mar 14, 2024

OFF HOLD

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Weekly KSv2 labels Mar 15, 2024
@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 labels Mar 22, 2024
@melvin-bot melvin-bot bot changed the title [QAB] Optimistically update the QAB NVP when taking actions [HOLD for payment 2024-03-29] [QAB] Optimistically update the QAB NVP when taking actions Mar 22, 2024
@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Mar 22, 2024
Copy link

melvin-bot bot commented Mar 22, 2024

Reviewing label has been removed, please complete the "BugZero Checklist".

Copy link

melvin-bot bot commented Mar 22, 2024

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.55-3 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

If no regressions arise, payment will be issued on 2024-03-29. 🎊

For reference, here are some details about the assignees on this issue:

  • @eVoloshchak requires payment through NewDot Manual Requests

Copy link

melvin-bot bot commented Mar 22, 2024

BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

  • [@eVoloshchak] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.
  • [@CortneyOfstad] Link the GH issue for creating/updating the regression test once above steps have been agreed upon.

@CortneyOfstad
Copy link
Contributor

@eVoloshchak can you clarify whether or not this needs a regression test by EOD today?

@CortneyOfstad
Copy link
Contributor

Payment Summary

@eVoloshchak to be paid $500 via NewDot

@eVoloshchak
Copy link
Contributor

Regression Test Proposal

  1. Log into newDot
  2. Request money from someone and choose manual.
  3. Open the Console and go to Application > IndexedDB > OnyxDB > keyValuePairs
  4. Confirm nvp_quickActionGlobalCreate is set to requestManual
  5. Repeat steps 1-4, but choosing Scan. Confirm nvp_quickActionGlobalCreate is set to requestScan
  6. Repeat steps 1-4, but choosing Distance. Confirm nvp_quickActionGlobalCreate is set to requestDistance
  7. Repeat steps 1-4, but splitting the request between several people. Confirm nvp_quickActionGlobalCreate is set to split Manual.

Do we agree 👍 or 👎

@CortneyOfstad
Copy link
Contributor

Regression test created here — https://github.com/Expensify/Expensify/issues/384210

@github-project-automation github-project-automation bot moved this from Release 1: Spring 2024 (May) to Done in [#whatsnext] #wave-collect Apr 1, 2024
@JmillsExpensify
Copy link

$500 approved for @eVoloshchak based on summary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Engineering Internal Requires API changes or must be handled by Expensify staff NewFeature Something to build that is a new item. Weekly KSv2
Projects
No open projects
Archived in project
Development

No branches or pull requests

4 participants