Skip to content

Payment issue for E/App PR#61228 #61252

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.

Sign up for GitHub

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

Jump to bottom
Closed
melvin-bot bot opened this issue May 1, 2025 · 8 comments
Closed

Payment issue for E/App PR#61228 #61252

melvin-bot bot opened this issue May 1, 2025 · 8 comments
Assignees
@jliexpensify @c3024
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Internal Requires API changes or must be handled by Expensify staff Task

Comments

@melvin-bot
Copy link

melvin-bot bot commented May 1, 2025

Issue created to compensate the Contributor+ member for their work on #61228
E/E issue linked to the PR - no internal issue.
Contributor+ member who reviewed the PR - @c3024.

Wait 7 days after the PR is deployed to production before issuing payment in case there are regressions.
@melvin-bot melvin-bot bot added Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Task Weekly KSv2 labels May 1, 2025
@melvin-bot melvin-bot bot added the Internal Requires API changes or must be handled by Expensify staff label May 1, 2025
@melvin-bot melvin-bot bot mentioned this issue May 1, 2025
Merged
51 tasks
@melvin-bot Melvin Bot
Copy link
Author

melvin-bot bot commented May 1, 2025

Triggered auto assignment to @jliexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels May 1, 2025
@jliexpensify
Copy link
Contributor

@c3024 does this count as a regression?

@c3024
Copy link
Contributor

c3024 commented May 2, 2025

Yes, it does although I feel it's a bit difficult to catch in normal C+ testing.

@jliexpensify
Copy link
Contributor

jliexpensify commented May 2, 2025 •
edited
Loading

Payment Summary

  • C+: @c3024 $125 (to be paid via New Expensify Requests)

@melvin-bot melvin-bot bot added the Overdue label May 5, 2025
@melvin-bot Melvin Bot
Copy link
Author

melvin-bot bot commented May 5, 2025

@jliexpensify, @c3024 Whoops! This issue is 2 days overdue. Let's get this updated quick!

@c3024
Copy link
Contributor

c3024 commented May 5, 2025

Not overdue.

@melvin-bot melvin-bot bot added Daily KSv2 and removed Overdue Daily KSv2 labels May 5, 2025
@jliexpensify
Copy link
Contributor

Deployed to prod 5 days ago

@jliexpensify
Copy link
Contributor

Payment Summary

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jliexpensify jliexpensify

@c3024 c3024

Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Internal Requires API changes or must be handled by Expensify staff Task
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jliexpensify @c3024