-
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
Web - Room - Paid policy isn't shown as an option under “Workspace”. #28046
Comments
Triggered auto assignment to @bfitzexpensify ( |
Bug0 Triage Checklist (Main S/O)
|
Reproduced. @jasperhuangg, any insight on this one? It was discovered via #26982 |
Bump on #28046 (comment) @jasperhuangg |
@bfitzexpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@jasperhuangg thoughts on #28046 (comment)? |
Another bump here @jasperhuangg |
@bfitzexpensify this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks! |
@jasperhuangg not sure if you're seeing my bumps, so assigning it to you so it appears in your K2. Looking for your thoughts on #28046 (comment) |
Ah, thanks for assigning me @bfitzexpensify. I wasn't receiving mention notifications for some reason. Taking a look. |
The QAer didn't follow the steps correctly. The policyID they entered has carets around it In addition, I'm noticing some other problems, fixing those in a PR |
Sweet, thanks man |
@jasperhuangg, @bfitzexpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@jasperhuangg, @bfitzexpensify Huh... This is 4 days overdue. Who can take care of this? |
@jasperhuangg, @bfitzexpensify 8 days overdue is a lot. Should this be a Weekly issue? If so, feel free to change it! |
@jasperhuangg, @bfitzexpensify 10 days overdue. Is anyone even seeing these? Hello? |
@jasperhuangg, @bfitzexpensify 12 days overdue now... This issue's end is nigh! |
Looks like https://github.com/Expensify/Web-Expensify/pull/39143 was deployed to prod a couple of weeks ago - I can't reproduce any more, so I think we're good to close this. |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Issue found wen executing PR #26982
Action Performed:
const policyID = '';
API.policy_get({policyIDList: policyID}).then((response) => {
const policy = response.policies[policyID];
policy.areChatRoomsEnabled = true;
API.policy_save({policyID, value: policy});
Expected Result:
Verify the paid policy shows up as an option under "Workspace"
Actual Result:
Paid policy isn't shown as an option under “Workspace”
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.73.0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Bug6210945_26982_web.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: