-
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: versioned segment override change request #3790
feat: versioned segment override change request #3790
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Uffizzi Preview |
# Conflicts: # frontend/common/types/requests.ts # frontend/web/components/modals/CreateFlag.js
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.
There's an outstanding bug in the API which is resolved here but this PR seems sound.
I've also added one minor suggestion which we should probably add before merging.
Co-authored-by: Matthew Elwell <[email protected]>
# Conflicts: # frontend/common/types/responses.ts
…change-request' into feat/versioned-segment-override-change-request
@kyle-ssg I've just noticed that the change request view doesn't actually show which segment a feature is being overridden by? ![]() |
@matthewelwell both issues should be resolved now |
@kyle-ssg when you create a CR with more than one segment override, the priority values don't seem right. ![]() |
@kyle-ssg I can't create any change requests now with segment overrides following the latest change: ![]() |
Related #1204 |
@kyle-ssg Did we decide to explicitly exclude scheduling changes on segment overrides on environments without change requests enabled? Editing a feature's state without change requests enabled let you schedule changes: In comparison, editing segment overrides without change requests only let you make an immediate update: |
Thanks for submitting a PR! Please check the boxes below:
pre-commit
to check lintingdocs/
if required so people know about the feature!Changes
Environments with versioning + Change Requests enabled will now have the ability to create a change request for segment overrides

Change requests now diff all of the feature states in comparison to live, irrespective of if versioning is enabled.
Much like versioning this will tell you changed, created and unchanged values
The create feature modal toast will now differentiate when a change request was created vs editing a feature. Creating a change request will close the create feature modal
Removed a confusing unused field "hide_from_client" from feature states
Note: versioned change requests cannot be edited due to the limited use case and complexity of updating a versions feature states in a change request.
How did you test this code?