-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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-07-24] [$50] Rip out the beta code for REPORT_FIELDS #44621
Comments
Job added to Upwork: https://www.upwork.com/jobs/~0108e03ca805bf2401 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @c3024 ( |
Upwork job price has been updated to $50 |
ProposalPlease re-state the problem that we are trying to solve in this issue.Remove the beta code for REPORT_FIELDS What is the root cause of that problem?Feature Request What changes do you think we should make in order to solve the problem?We will remove the code corresponding to Lines 18 to 20 in 0c8bae5
|
@neonbhai can be assigned. 🎀 👀 🎀 C+ Reviewed |
Current assignee @thienlnam is eligible for the choreEngineerContributorManagement assigner, not assigning anyone new. |
📣 @c3024 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app! |
📣 @neonbhai 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
@thienlnam, @neonbhai, @c3024 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.7-8 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-07-24. 🎊 For reference, here are some details about the assignees on this issue:
|
Issue is ready for payment but no BZ is assigned. @bfitzexpensify you are the lucky winner! Please verify the payment summary looks correct and complete the checklist. Thanks! |
@thienlnam, @bfitzexpensify, @neonbhai, @c3024 Whoops! This issue is 2 days overdue. Let's get this updated quick! |
cc @bfitzexpensify Could we handle payment here? No regression tests needed for this |
Payments complete. Closing this out |
We're no longer using the code for CONST.BETAS.REPORT_FIELDS. This is enabled for everyone internally - so we can just remove the beta code related to CONST.BETAS.REPORT_FIELDS
Note: This is different from the beta REPORT_FIELDS_FEATURE
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @bfitzexpensifyThe text was updated successfully, but these errors were encountered: