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

Payment issue for E/App PR#56891 #57561

Open
melvin-bot bot opened this issue Feb 27, 2025 · 4 comments
Open

Payment issue for E/App PR#56891 #57561

melvin-bot bot opened this issue Feb 27, 2025 · 4 comments
Assignees
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 Reviewing Has a PR in review Task

Comments

@melvin-bot
Copy link

melvin-bot bot commented Feb 27, 2025

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

Wait 7 days after the PR is deployed to production before issuing payment in case there are regressions.

Issue OwnerCurrent Issue Owner: @twisterdotcom
@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 Feb 27, 2025
@melvin-bot melvin-bot bot added the Internal Requires API changes or must be handled by Expensify staff label Feb 27, 2025
@melvin-bot melvin-bot bot mentioned this issue Feb 27, 2025
50 tasks
Copy link
Author

melvin-bot bot commented Feb 27, 2025

Triggered auto assignment to @twisterdotcom (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 Feb 27, 2025
@twisterdotcom
Copy link
Contributor

twisterdotcom commented Feb 28, 2025

Sure we could create these when it's deployed, or 7 days later even but whatever. I'll keep an eye on it @parasharrajat. I assume this is the standard $250?

cc @roryabraham

@twisterdotcom twisterdotcom added the Reviewing Has a PR in review label Feb 28, 2025
@roryabraham
Copy link
Contributor

standard $250, yes. I agree we should create these at least when the PR is merged (technically easier than when it's deployed or 7 days later), but it's maybe not worth the time to fix it since PRs not attached to issues are an edge case

@twisterdotcom
Copy link
Contributor

Merged, but not deployed yet. Switching to Weekly for the time being. Will try my best to stay on top of this @parasharrajat.

@twisterdotcom twisterdotcom added Weekly KSv2 and removed Daily KSv2 labels Mar 3, 2025
@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Mar 6, 2025
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 Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review Task
Projects
None yet
Development

No branches or pull requests

3 participants