Skip to content

Versions environments updates #4949

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.

Sign up for GitHub

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

Jump to bottom
Open
tdgao wants to merge 27 commits into
base: main
Choose a base branch
from
Open

Versions environments updates #4949

tdgao wants to merge 27 commits into from

Conversation

@tdgao
Copy link
Contributor

@tdgao tdgao commented Dec 22, 2025 •
edited
Loading

Changes:

  • Show project settings environment page only for staff members. Show a redirect message for non-staff users.
  • Open environment migration modal in alert.
  • If a project has different version environments, then it will show the environment column in versions table.

@tdgao tdgao changed the title Additional version environments updates Versions environments updates Dec 24, 2025
@tdgao tdgao marked this pull request as ready for review December 24, 2025 19:24
@tdgao tdgao requested review from IMB11 and Prospector December 24, 2025 19:26
IMB11
IMB11 requested changes Dec 24, 2025
View reviewed changes
Copy link
Member

@IMB11 IMB11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good, just two things

@IMB11
Copy link
Member

IMB11 commented Dec 24, 2025

Also, was the TriangleAlertIcon warning removal from /dashboard/projects intentional?

@tdgao tdgao requested a review from IMB11 December 24, 2025 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@IMB11 IMB11 IMB11 approved these changes

@Prospector Prospector Awaiting requested review from Prospector

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tdgao @IMB11