Skip to content

Fix edit permissions only checking the root permission#113

Merged
woodwoerk merged 1 commit into
mainfrom
fix/edit-permissions
May 20, 2026
Merged

Fix edit permissions only checking the root permission#113
woodwoerk merged 1 commit into
mainfrom
fix/edit-permissions

Conversation

@woodwoerk

Copy link
Copy Markdown
Collaborator

Description

Update the permissions handling logic to check if the user has edit permissions for the selected plan. Previously (in staging, not production) we incorrectly checked the edit permissions on the root NZC framework, which was only true for superusers.


✅ Pre-Merge Checklist

Type of Change

  • Set the PR's label to match the nature of this change

Testing

  • Built Unit tests (unit tests added/updated)
  • Authorization is tested (permissions and access controls verified)
  • Mobile screen widths tested for responsiveness (if applicable)
  • Manually tested locally in all affected projects (functionality verified)
    Manual testing instructions
    If feature requires manual testing by reviewer, you can provide instructions here.

Internationalization & Accessibility

  • Accessibility standards met (WCAG compliance, screen reader support)

Dependencies

  • Dependencies are merged (if applicable. If the change depends on other PRs)

@woodwoerk woodwoerk requested a review from lilia1891 May 18, 2026 06:11
@woodwoerk woodwoerk added the Bug fix Bug fix (non-breaking change which fixes an issue) label May 18, 2026

@lilia1891 lilia1891 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good!

@woodwoerk woodwoerk merged commit cd8ad26 into main May 20, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug fix Bug fix (non-breaking change which fixes an issue)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants