Skip to content

Assessment Module Rule Update - #2938

Merged
adelowo merged 6 commits into
mainfrom
feat-ISS-3322
Oct 26, 2026
Merged

adelowo merged 6 commits into
mainfrom
feat-ISS-3322

Conversation

@adelowo

@adelowo adelowo commented Oct 26, 2026 •

Copy link
Copy Markdown
Member

enforces modules based off the type of custom domain being created. prevents an issue where orgs having only one module type may grant them access to all available domain types

also task regenerate

@adelowo
adelowo marked this pull request as ready for review October 26, 2026 09:52
@adelowo
adelowo requested a review from a team as a code owner October 26, 2026 09:52
@github-actions github-actions Bot added the cli label Oct 26, 2026
@adelowo
adelowo enabled auto-merge (squash) October 26, 2026 12:26
@github-actions github-actions Bot removed the cli label Oct 26, 2026
}

func getPurposesFromMutation(ctx context.Context, m *generated.CustomDomainMutation) ([]enums.DomainPurpose, error) {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should only need this on create because if the addon is removed, we'd want to delete the custom domain.

@sonarqubecloud

Copy link
Copy Markdown

@adelowo
adelowo merged commit 7beddf4 into main Oct 26, 2026
24 checks passed
@adelowo
adelowo deleted the feat-ISS-3322 branch October 26, 2026 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants