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

Access permissions 3: Smaller refactorings #6879

Merged
merged 2 commits into from
Jan 20, 2025

Conversation

lukasbestle
Copy link
Member

Description

Summary of changes

Smaller refactorings

Reasoning

Consistency and cleanup

Changelog

Housekeeping

  • Clean up permission-related code for consistency

Breaking changes

None

Docs

None

Ready?

  • In-code documentation (wherever needed)
  • Unit tests for fixed bug/feature
  • Tests and CI checks all pass

For review team

  • Add lab and/or sandbox examples (wherever helpful)
  • Add changes & docs to release notes draft in Notion

@lukasbestle lukasbestle force-pushed the v5/feature/access-permissions-3 branch 2 times, most recently from ccdbb89 to 936a49f Compare January 18, 2025 16:42
@lukasbestle lukasbestle requested a review from a team January 18, 2025 16:42
@lukasbestle lukasbestle force-pushed the v5/feature/access-permissions-3 branch from 936a49f to 27ace7a Compare January 18, 2025 16:45
@bastianallgeier bastianallgeier merged commit 32df7b4 into v5/develop Jan 20, 2025
12 checks passed
@bastianallgeier bastianallgeier deleted the v5/feature/access-permissions-3 branch January 20, 2025 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants