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-06-13] CRITICAL: GBR and Settlement button for the receiver on the invoice report preview #40437

Closed
cristipaval opened this issue Apr 18, 2024 · 21 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 NewFeature Something to build that is a new item.

Comments

@cristipaval
Copy link
Contributor

cristipaval commented Apr 18, 2024

Part of the Invoicing V0.2 project

Main issue: https://github.com/Expensify/Expensify/issues/341717
Doc section: Invoicing V1
Project: #vip-billpay

Feature Description

Implement the Settlement button on the invoice report preview in the invoice room, when the user is the individual receiver.

Manual Test Steps

Automated Tests

Issue OwnerCurrent Issue Owner: @trjExpensify
@cristipaval cristipaval added Daily KSv2 NewFeature Something to build that is a new item. labels Apr 18, 2024
@cristipaval cristipaval self-assigned this Apr 18, 2024
@cristipaval cristipaval changed the title CRITICAL: GBR and Settlement button for the receiver on the invoice report preview [HOLD] CRITICAL: GBR and Settlement button for the receiver on the invoice report preview Apr 18, 2024
Copy link

melvin-bot bot commented Apr 18, 2024

Triggered auto assignment to @trjExpensify (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Apr 18, 2024
@cristipaval cristipaval moved this to Critical in [#whatsnext] #billpay Apr 18, 2024
@trjExpensify
Copy link
Contributor

Feature Description

Implement the Settlement button on the

😅 Heads up, man! I think you forgot to finish your sen

@cristipaval
Copy link
Contributor Author

DANG! Sorry

@melvin-bot melvin-bot bot added the Overdue label Apr 26, 2024
@trjExpensify
Copy link
Contributor

👋 @cristipaval what issue is this on hold for? We should update the title to make that dependency clear.

@melvin-bot melvin-bot bot removed the Overdue label Apr 27, 2024
@cristipaval
Copy link
Contributor Author

this is part of the V0.2 which is held until the v0.1 is launched

@waterim
Copy link
Contributor

waterim commented Apr 29, 2024

@cristipaval Should I take this asap as v1 was merged?

@cristipaval cristipaval changed the title [HOLD] CRITICAL: GBR and Settlement button for the receiver on the invoice report preview CRITICAL: GBR and Settlement button for the receiver on the invoice report preview May 6, 2024
@cristipaval
Copy link
Contributor Author

@waterim yes please start working on this when the critical issues on the V0.1 are cleared

@cristipaval cristipaval added Daily KSv2 and removed Weekly KSv2 labels May 6, 2024
@cristipaval
Copy link
Contributor Author

cristipaval commented May 7, 2024

^Auth PR is ready for review. It enables the GBR to the receiver on the invoice room in the LHN.

@cristipaval
Copy link
Contributor Author

We have a draft App PR^ 🙌

@VickyStash
Copy link
Contributor

Updates:

  • addressed designers' feedback
  • waiting for C+ review

@melvin-bot melvin-bot bot added Monthly KSv2 and removed Weekly KSv2 labels Jun 3, 2024
Copy link

melvin-bot bot commented Jun 3, 2024

This issue has not been updated in over 15 days. @trjExpensify, @cristipaval, @waterim eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

@trjExpensify trjExpensify added Weekly KSv2 and removed Monthly KSv2 labels Jun 3, 2024
@trjExpensify
Copy link
Contributor

PR is in active review, Melv.

Copy link

melvin-bot bot commented Jun 5, 2024

⚠️ 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 melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 labels Jun 6, 2024
@melvin-bot melvin-bot bot changed the title CRITICAL: GBR and Settlement button for the receiver on the invoice report preview [HOLD for payment 2024-06-13] CRITICAL: GBR and Settlement button for the receiver on the invoice report preview Jun 6, 2024
Copy link

melvin-bot bot commented Jun 6, 2024

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

@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Jun 6, 2024
Copy link

melvin-bot bot commented Jun 6, 2024

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.79-11 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-06-13. 🎊

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

  • @waterim does not require payment (Contractor)

Copy link

melvin-bot bot commented Jun 6, 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:

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

@ikevin127
Copy link
Contributor

The regression message from here is false alarm and sorted out in #43145 (comment).

Regarding #40437 (comment), I reviewed PR as C+ and require compensation.

Note: Given the status of the issue is Critical, the bounty should be $500.

cc @trjExpensify

@trjExpensify
Copy link
Contributor

Assigned you to the issue to not lose track of that.

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Jun 12, 2024
@trjExpensify
Copy link
Contributor

@ikevin127 offer sent for $500 for the C+ review.

@ikevin127
Copy link
Contributor

ikevin127 commented Jun 13, 2024

@trjExpensify Offer accepted (awaiting payment), thank you!

Copy link

melvin-bot bot commented Jun 13, 2024

Payment Summary

Upwork Job

BugZero Checklist (@trjExpensify)

  • I have verified the correct assignees and roles are listed above and updated the neccesary manual offers
  • I have verified that there are no duplicate or incorrect contracts on Upwork for this job (https://www.upwork.com/ab/applicants//hired)
  • I have paid out the Upwork contracts or cancelled the ones that are incorrect
  • I have verified the payment summary above is correct

@twisterdotcom twisterdotcom self-assigned this Jun 14, 2024
@github-project-automation github-project-automation bot moved this from Critical to Done in [#whatsnext] #billpay Jun 14, 2024
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 Daily KSv2 NewFeature Something to build that is a new item.
Projects
Status: Done
Development

No branches or pull requests

6 participants