[HOLD for payment 2023-12-06] [ECARD] CRITICAL: Transactions - Enable MoneyRequestView Edit Transactions Logic

[grgia]

Design Doc Section

https://docs.google.com/document/d/1iELOXPOAcnuIPZJ3XbO1S8_3qHaM31Ptf8dR4LRZMyE/edit#bookmark=id.qpl4zgrq5w2x

Overview

Goal:
We need to Hold on updates to EditMoneyRequest to handle card transactions. Once that’s complete, we will add the edit logic for non reimbursable transactions to NewDot.

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~0149be352f2615a7b7
• Upwork Job ID: 1709592009508114432
• Last Price Increase: 2023-10-04

[melvin-bot]

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

[melvin-bot]

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

[melvin-bot]

Triggered auto assignment to @Christinadobrzyn (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details.

[grgia]

HOLD on edit functionality
cc @luacmartins

[Christinadobrzyn]

This is on Hold still

[grgia]

still on hold

[grgia]

on hold still, but close!

[melvin-bot]

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

[melvin-bot]

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

[Christinadobrzyn]

Looks like the blocker label has been removed from #32077 and @grgia is working on a solution.

I think we're good to continue working on this job!

[melvin-bot]

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

[melvin-bot]

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

• Allow editing card transactions (not amount) #31353

If no regressions arise, payment will be issued on 2023-12-06. 🎊

After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

• External issue reporter
• Contributor that fixed the issue
• Contributor+ that helped on the issue and/or PR

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

• @akinwale requires payment

[melvin-bot]

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

• [@akinwale] The PR that introduced the bug has been identified. Link to the PR:
• [@akinwale] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
• [@akinwale] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
• [@akinwale] Determine if we should create a regression test for this bug.
• [@akinwale] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
• [@Christinadobrzyn] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

[Christinadobrzyn]

prepping for payment!

Payouts due:

Issue Reporter: NA
Contributor: NA
Contributor+: $500 @akinwale (in Upwork)

Eligible for 50% #urgency bonus? N

Upwork job is here.

[codewithnimesh]

Design Doc Section

https://docs.google.com/document/d/1iELOXPOAcnuIPZJ3XbO1S8_3qHaM31Ptf8dR4LRZMyE/edit#bookmark=id.qpl4zgrq5w2x

Overview

Goal: We need to Hold on updates to EditMoneyRequest to handle card transactions. Once that’s complete, we will add the edit logic for non reimbursable transactions to NewDot.

Upwork Automation - Do Not Edit

I checked your job posted on Upwork https://www.upwork.com/jobs/~0178d694337026c11f , FYI its private job i.e, its details are not visible in upwork. You may need to correct the access while generating JOB via bot :)

Hi, I'm in interested in the project, let me know when we can have a quick discussion call?

[melvin-bot]

📣 @codewithnimesh! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

1. Make sure you've read and understood the contributing guidelines.
2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
   
   Format:

Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

[Christinadobrzyn]

Hi @codewithnimesh! Thanks for reaching out - this job is complete but feel free to apply to any jobs that have a Help Wanted label in our Expensify/App repo - https://github.com/Expensify/App/issues?q=is%3Aopen+is%3Aissue+label%3A%22Help+Wanted%22

[Christinadobrzyn]

@akinwale I paid you in Upwork based on the payment structure here - #28836 (comment)

Closing this but let me know if I'm missing anything!