Skip to content

Fix: Adds correct environmentSlug#127

Merged
CGoodwin90 merged 2 commits intomainfrom
pending-deploy-issue
Feb 4, 2026
Merged

Fix: Adds correct environmentSlug#127
CGoodwin90 merged 2 commits intomainfrom
pending-deploy-issue

Conversation

@CGoodwin90
Copy link
Collaborator

@CGoodwin90 CGoodwin90 commented Feb 4, 2026

Corrects the slug passed to usePendingChangesNotification in environment routes. Updates Routes to pass the environmentSlug as a prop to EnvironmentRoutesPage to be consistent with other Environment components.

shreddedbacon
shreddedbacon approved these changes Feb 4, 2026
View reviewed changes
Copy link
Member

@shreddedbacon shreddedbacon left a comment

Choose a reason for hiding this comment

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

Works as expected

@CGoodwin90 CGoodwin90 merged commit 0fdf8ba into main Feb 4, 2026
1 check passed
@CGoodwin90 CGoodwin90 deleted the pending-deploy-issue branch February 4, 2026 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shreddedbacon shreddedbacon shreddedbacon approved these changes

@mattswann mattswann Awaiting requested review from mattswann

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@CGoodwin90 @shreddedbacon