Skip to content
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

Remove worldwide organisation field from role edit view when feature flag enabled #9043

Merged
merged 1 commit into from
May 15, 2024

Conversation

brucebolt
Copy link
Member

For Editionable Worldwide Organisations, we are assigning roles to them through the editionable user interface and through the EditionRole relationship.

However the non-editionable version of Worldwide Organisations has them assigned on the role user interface and through the WorldwideOrganisationRole relationship.

Therefore disabling the non-editionable way of assigning roles when the editionable worldwide organisation feature flag is enabled.

Trello card

@brucebolt brucebolt changed the title Remove worldwide organisation field when feature flag enabled Remove worldwide organisation field from role edit view when feature flag enabled May 15, 2024
@brucebolt brucebolt force-pushed the remove-legacy-role-ww-org branch 2 times, most recently from 73fb5f4 to ee249d7 Compare May 15, 2024 08:03
For Editionable Worldwide Organisations, we are assigning roles to them
through the editionable user interface and through the `EditionRole`
relationship.

However the non-editionable version of Worldwide Organisations has them
assigned on the role user interface and through the
`WorldwideOrganisationRole` relationship.

Therefore disabling the non-editionable way of assigning roles when the
editionable worldwide organisation feature flag is enabled.
@brucebolt brucebolt marked this pull request as ready for review May 15, 2024 08:33
@brucebolt brucebolt merged commit 819bd15 into main May 15, 2024
23 checks passed
@brucebolt brucebolt deleted the remove-legacy-role-ww-org branch May 15, 2024 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants