Skip to content

[Due for payment 2025-04-28] [GR Phase 4] Glue together Connect Bank Account screen (pre-step 1) with freshly created frame #50912

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

Closed
madmax330 opened this issue Oct 16, 2024 · 14 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2

Comments

@madmax330
Copy link
Contributor

Design doc section: https://docs.google.com/document/d/1Y4P7G9h0Easjcea7wyk52gKrNB9o1yTjpmJFYkrHDW4/edit?tab=t.0#bookmark=kix.dka6v1ph5j6s

Copy link

melvin-bot bot commented Jan 3, 2025

@madmax330, this Monthly task hasn't been acted upon in 6 weeks; closing.

If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead.

Copy link

melvin-bot bot commented Apr 19, 2025

⚠️ 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.

Copy link

melvin-bot bot commented Apr 21, 2025

⚠️ 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 Apr 21, 2025
@melvin-bot melvin-bot bot changed the title [GR Phase 4] Glue together Connect Bank Account screen (pre-step 1) with freshly created frame [Due for payment 2025-04-28] [GR Phase 4] Glue together Connect Bank Account screen (pre-step 1) with freshly created frame Apr 21, 2025
Copy link

melvin-bot bot commented Apr 21, 2025

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

@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Apr 21, 2025
Copy link

melvin-bot bot commented Apr 21, 2025

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.30-4 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 2025-04-28. 🎊

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

  • @hungvu193 requires payment through NewDot Manual Requests
  • @MrMuzyk does not require payment (Contractor)

@Christinadobrzyn
Copy link
Contributor

Christinadobrzyn commented Apr 22, 2025

Hi there, I have this GH that I think is related to this project - #60570.

I can't tell if it's a regression or something new? Should this be fixed separately?

cc @hungvu193 and @madmax330

@hungvu193
Copy link
Contributor

Hi there, I have this GH that I think is related to this project - #60570.

I can't tell if it's a regression or something new? Should this be fixed separately?

cc @hungvu193 and @madmax330

Thanks for pinging here @Christinadobrzyn. Yes, GR is under beta, and anything related will be fixed in a follow-up issue here:
#60633

@melvin-bot melvin-bot bot removed the Weekly KSv2 label Apr 28, 2025
@melvin-bot melvin-bot bot added the Daily KSv2 label Apr 28, 2025
Copy link

melvin-bot bot commented Apr 28, 2025

Issue is ready for payment but no BZ is assigned. @strepanier03 you are the lucky winner! Please verify the payment summary looks correct and complete the checklist. Thanks!

Copy link

melvin-bot bot commented Apr 28, 2025

Payment Summary

Upwork Job

  • Reviewer: @hungvu193 owed $250 via NewDot
  • Contributor: @MrMuzyk is from an agency-contributor and not due payment

BugZero Checklist (@strepanier03)

  • I have verified the correct assignees and roles are listed above and updated the necessary 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

@strepanier03
Copy link
Contributor

@hungvu193 - Before I post the payment summary, can you confirm if we need a reg test for this?

@hungvu193
Copy link
Contributor

The PR only hides this feature under beta, so I don't think we need regression test for this one

@hungvu193 - Before I post the payment summary, can you confirm if we need a reg test for this?

@melvin-bot melvin-bot bot added the Overdue label May 1, 2025
@strepanier03
Copy link
Contributor

Payment Summary

@melvin-bot melvin-bot bot removed the Overdue label May 1, 2025
@strepanier03
Copy link
Contributor

@hungvu193 payment summary above, feel free to request!

@garrettmknight
Copy link
Contributor

$250 approved for @hungvu193

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
Projects
None yet
Development

No branches or pull requests

6 participants