-
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-11-20] [A/B Testing][Combined Track Submit] Define logic for displaying test to half of users #49591
Comments
https://github.com/Expensify/App/blob/main/src/libs/Permissions.ts#L48-L52 Thinking we just remove the beta and then add the even/odd logic here instead- we could even create a new |
Can you discuss your proposed solution in #engineering-chat tomorrow and make sure we're not missing anything? Then we can move forward with this. I just don't know enough to spot if there's any concerns with this route. |
I can open a PR for this this week |
PR in review |
@grgia Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@grgia Still overdue 6 days?! Let's take care of this! |
@grgia 8 days overdue is a lot. Should this be a Weekly issue? If so, feel free to change it! |
@grgia 12 days overdue. Walking. Toward. The. Light... |
This issue has not been updated in over 14 days. @grgia eroding to Weekly issue. |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.60-3 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-11-20. 🎊 |
All internal, closing out as complete |
Show combinedTrackSubmit to 1/2 of all users
https://expensify.slack.com/archives/C07HPDRELLD/p1726854156845179?thread_ts=1726506289.927939&cid=C07HPDRELLD
The text was updated successfully, but these errors were encountered: