Skip to content

feat: create and edit environment required approvals #9621

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

Merged
merged 4 commits into from
Mar 26, 2025

Conversation

kwasniew
Copy link
Contributor

@kwasniew kwasniew commented Mar 26, 2025

About the changes

Can create and edit environment with required change request approvals
Screenshot 2025-03-26 at 14 34 19

Details:

  • change behind a flag
  • current update needs to distinguish between ignoring field in update (undefined) and reseting field to null in update

Important files

Discussion points

Copy link

vercel bot commented Mar 26, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
unleash-docs ⬜️ Ignored (Inspect) Visit Preview Mar 26, 2025 1:39pm

Copy link
Contributor

github-actions bot commented Mar 26, 2025

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

@@ -198,7 +198,7 @@ export interface IEnvironment {
projectCount?: number;
apiTokenCount?: number;
enabledToggleCount?: number;
requiredApprovals?: number;
requiredApprovals?: number | null;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

we have to distinguish null and undefined. undefined means skip this field in update and keep the previous value, null means reset this field to null in update

<StyledRequiredApprovals>
Required approvals
</StyledRequiredApprovals>
<GeneralSelect
Copy link
Contributor Author

Choose a reason for hiding this comment

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

copied from the table with change request settings

marginBottom: theme.spacing(0.5),
}));

const useApprovalOptions = () => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

copied from change requests table

@kwasniew kwasniew requested a review from FredrikOseberg March 26, 2025 13:40
@kwasniew kwasniew merged commit 1bd328f into main Mar 26, 2025
11 checks passed
@kwasniew kwasniew deleted the create-edit-environment-required-approvals branch March 26, 2025 14:54
@github-project-automation github-project-automation bot moved this from New to Done in Issues and PRs Mar 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant