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
E2C: Migration summary #84890
E2C: Migration summary #84890
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks great 🥳 just nitpicking really 😄
94d3c85
to
a15bc5f
Compare
alignItems="center" | ||
justifyContent="space-between" | ||
> | ||
<Stack gap={4} alignItems="center"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<Stack gap={4} alignItems="center"> |
This Stack is not needed anymore.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hm yeah you're technically right for the current state, I guess 😅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i sympathise cause i know from the design this will be needed fairly imminently 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 to alex's comment, otherwise lgtm 👍
Adds the Migration Summary box to the top of Migrate to Cloud to show the current connected stack, disconnect, and to start the migration.
startMigration
API mutation. It doesn't really do anything, not even a mocked behaviour. I figured this is more dependant on real functionality, so I think this is fine for now.Fixes #84553