-
Notifications
You must be signed in to change notification settings - Fork 429
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
feat: Deletes provider and displays warning in selected env #5085
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
Docker builds report
|
Uffizzi Preview |
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.
This looks good to me, however, I feel that the environment-level warning is too buried at the moment. A user would need to open the dropdown to see them.
Can we highlight the dropdown somehow to direct users attention to the dropdown in case when one of the not currently selected environments is unhealthy?
@tiagoapolo, sorry, there's a pretty minor, but rather annoying styling issue with the tooltip here when appearing over an environment banner. ![]() Otherwise, this all looks good to me now. |
Indeed, that happens mainly because we need a more consistent z-index value management for handling component hierarchy. Ideally we want to render tooltips and alike in portals. This PR will fix that issue. |
bbedf58
to
b2462fd
Compare
small comment other than that all good 👍 |
Thanks for submitting a PR! Please check the boxes below:
docs/
if required so people know about the feature!Changes
Ref: #3615
Environment without unhealthy features example

How did you test this code?
Selected env warning
Delete a provider