fix!: require org membership for user ACLs (#26852) - #27884
Conversation
|
👋 Hey @jdomeracki-coder! This PR is targeting the Only bug fixes should be cherry-picked to release branches. If this is a bug fix, please update the PR title to match the conventional commit format: If this is not a bug fix, it likely should not target a release branch. |
mtojek
left a comment
mtojek
left a comment
There was a problem hiding this comment.
The original PR has been marked as introducing breaking changes, so we will introduce a breaking on into a patch release -> semver violations.
In principle we shouldn't do this, so can we confirm this is expected behavior?
Good callout, let's discuss on Slack |
mtojek
left a comment
mtojek
left a comment
There was a problem hiding this comment.
Discussed over slack and agreed to ship 👍
|
As discussed internally, this patch is good to be backported to ESR |
3 participants
Backport of #26852
Original PR: #26852 — fix!: require org membership for user ACLs
Merge commit: 6b3341a
Requested by: @jdomeracki-coder
Clean cherry-pick, no conflicts.
Opened by Coder Agents on behalf of @jdomeracki-coder.