-
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-01-02] Hook up with the Downgrade Workspace API #51515
Comments
Triggered auto assignment to @sonialiap ( |
|
Triggered auto assignment to Design team member for new feature review - @shawnborton ( |
This will be the final issue to complete, so it needs to HOLD on all the other issues from the project. Will assign contributor then. |
I think we're still holding on this one, right? |
Will try to make progress today. |
We are close to being able to do this, final BE PR is in review and we can start testing FE and BE together. |
Hey @DylanDylann, this should be ready to tackle now that the BE is complete :) |
@DylanDylann this feature is ready for the FE part! Will you be able to handle this one? |
@carlosmiceli Yeah I already did the FE implementation locally with a branch built up from #53343. I will raise PR after #53343 is merged |
Great! |
I'm OOO Dec 16-20, I don't think this will need a BZ before I'm back, but if it does please reapply the bug label |
@carlosmiceli, @shawnborton, @sonialiap, @DylanDylann Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Not overdue, we're working on it :D |
@DylanDylann I think I found the issue with the duplicate action, but can we proceed using the first one while I try to fix it? |
Fixed, will be merged soon. The action that says |
Please ping me when the change is deployed |
It's now deployed. |
@carlosmiceli Worked well |
Fantastic, thanks sir! cc @garrettmknight I'll go over all the open issues tomorrow to see what's left to do, but anything outstanding from your end @DylanDylann ? |
Everything works well from me |
To confirm, this is still the plan, correct? |
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results. If a regression has occurred and you are the assigned CM follow the instructions here. If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future. |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.78-6 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-01-02. 🎊 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:
|
Part of Add Workspace Plan Downgrades to NewDot
Doc section
Steps to complete:
Issue Owner
Current Issue Owner: @sonialiapThe text was updated successfully, but these errors were encountered: