-
Notifications
You must be signed in to change notification settings - Fork 3k
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 2025-02-06] [HOLD for payment 2025-01-21] Add plan changes to Subscriptions #52588
Comments
Triggered auto assignment to @joekaufmanexpensify ( |
Still very excited for this one. |
Hoping to prioritize this week while things are quiet. |
Sweet - got a minute to work through some copy options for when the user is an admin on multiple workspaces here. |
Alright - OP is updated, now we're just waiting for the main issue to complete to open this one up. |
Quick clarification about the HOLD here:
|
@jayeshmangwani Hey Jayesh, here's another issue we need to work on to really complete the full upgrades/downgrades project. Seems like a natural progression for you to take over this issue too, let me know if there are any questions. I do have one question for you: we need to implement a check for whether the admin manages multiple workspaces or not. Do we have a way to check for that in the FE? Probably not, but thought I'd ask. If not, what kind of changes would you need from the BE (as in, what command or response would you like to see changed and how)? |
Yes, that sounds good. I will start working on this issue once we merge PR for the downgrade workspace UI and new downgrade API. @carlosmiceli @garrettmknight I will also need access to Figma, as I cannot find this section in the design doc. |
@carlosmiceli For the multiple workspaces admin front-end check, I think we already have a function that can be used here. Lines 391 to 393 in ec76adc
|
Boom! Looks like we're good here @garrettmknight 💪 |
Thanks 🙌 |
BE is merged 🙌 |
Small BE change requested by our contributors has both PRs in review, which should completely unblock @jayeshmangwani from completing the plan page work. |
Both BE PRs have been merged. |
@garrettmknight, currently, we show the text |
I think we can remove the 'this cannot be undone' warning - they'll see it when they downgrade the workspaces individually. Here's the upgrade copy:
|
Thanks for the copies |
@carlosmiceli, @garrettmknight, @jayeshmangwani Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@jayeshmangwani let me know when you've got that PR up. |
@garrettmknight @DylanDylann Opened a PR to update the copies, but we still need confirmation for the Spanish version, which we raised on Slack. |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.91-2 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 2025-02-06. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
No BZ necessary, adding QA for the project. |
@garrettmknight @carlosmiceli If there are no tasks anymore. I think we are ready to handle the payment for this entire project. |
Agreed with Dylan—there are no pending tasks for this project. |
Yep! On my todo list. Do you two happen to have a list of issues you completed handy? |
We have 6 issues created for this feature and all are done by 4 PRs. Additionally we have some other regression/polish issues that not calculated here
|
Part of the Workspace Downgrades project
Main issue: https://github.com/Expensify/Expensify/issues/399936
Project: Workspace Downgrades
Feature Description
Your plan
section of the Subscriptions page to show both plan types with the current, highest plan type the account is an admin of highlighted/selected.Copy for options above in this Figma here
Manual Test Steps
Single Workspace Upgrade
Single Workspace Downgrade (follow Single Workspace Upgrade steps first)
Multi Workspace Upgrade (follow Single Workspace Upgrade + Downgrade steps first)
Multi Workspace Upgrade (follow Multi Workspace Upgrade steps first)
Automated Tests
N/A
Issue Owner
Current Issue Owner: @garrettmknightThe text was updated successfully, but these errors were encountered: