-
Notifications
You must be signed in to change notification settings - Fork 3k
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-08-05] Payment issue for E/App PR#41962 #44874
Comments
Triggered auto assignment to @puneetlath ( |
@puneetlath, @ishpaul777 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Hasn't been deployed to prod yet. |
Can we please bump the price up here? The PR was a large enough and required extensive testing. Thanks! |
@puneetlath, @ishpaul777 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Discussed on Slack. $1k is fair here given the scope/scale of the PR. |
@ishpaul777 do we need new regression tests here? |
I dont think we need new test cases, there was no change in functionality for user but major changes on how we handle comment linking code wise. |
@puneetlath Please hold my payment until 7/22 i'll bump once i am ready to accept payment, we can move this to weekly for now |
Not overdue |
Payment Summary
BugZero Checklist (@puneetlath)
|
Sadly i'd still have to keep my payment on hold (Expected End of this month) i'll ping once i am ready to accept. Thanks for you patience. |
Payment Summary
BugZero Checklist (@puneetlath)
|
Payment Summary
BugZero Checklist (@puneetlath)
|
@ishpaul777 can I go ahead and pay this? |
yes we can go ahead with payment now Thanks for holding payment 🙇 |
@ishpaul777 is this contract the one for this issue? If so, it looks like it was already paid: https://www.upwork.com/nx/wm/workroom/38010569/overview |
It was for a different PR #45313 (comment) |
issue- #45553 |
Ok got it. Send you an offer: https://www.upwork.com/nx/wm/offer/103524448. Ping me here once you've accepted. |
We agreed this for $1k #44874 (comment) :) |
Thanks for reminding me. Paid out $1k. |
Thank you! |
Issue created to compensate the Contributor+ member for their work on #41962
E/E issue linked to the PR - no internal issue.
Contributor+ member who reviewed the PR - @ishpaul777.
Wait 7 days after the PR is deployed to production before issuing payment in case there are regressions.
The text was updated successfully, but these errors were encountered: