Skip to content

feat(BA-4876): add GQL types and resolver for permission update mutation#9724

Draft
fregataa wants to merge 3 commits intomainfrom
feat/BA-4876-permission-update-gql
Draft

feat(BA-4876): add GQL types and resolver for permission update mutation#9724
fregataa wants to merge 3 commits intomainfrom
feat/BA-4876-permission-update-gql

Conversation

@fregataa
Copy link
Member

@fregataa fregataa commented Mar 5, 2026

Summary

  • UpdatePermissionInput strawberry input type with to_updater() method
  • admin_update_permission resolver registered in schema
  • Export registrations across __init__.py chain
  • Unit tests for resolver (action dispatch, partial/full update, error propagation, field mapping)

Test plan

Resolves BA-4876 (GQL layer)

🤖 Generated with Claude Code

- UpdatePermissionInput with to_updater() method
- admin_update_permission resolver registered in schema
- Export registrations across __init__.py chain
- Unit tests for resolver

Depends on: feat/BA-4876-permission-update-mutation

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 5, 2026 17:30
@fregataa fregataa self-assigned this Mar 5, 2026
@fregataa fregataa added this to the 26.3 milestone Mar 5, 2026
@fregataa fregataa marked this pull request as draft March 5, 2026 17:39
fregataa and others added 2 commits March 6, 2026 02:42
The resolver is defined for schema purposes but implementation will be
wired once the backend service layer PR is merged.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@github-actions github-actions bot added size:L 100~500 LoC comp:manager Related to Manager component labels Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp:manager Related to Manager component size:L 100~500 LoC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants