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.

Sign up for GitHub

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

Jump to bottom

Split cvat/apps/iam/permissions.py into per-app permissions modules #7734

Merged
merged 4 commits into from
Apr 25, 2024
Merged

Split cvat/apps/iam/permissions.py into per-app permissions modules #7734

merged 4 commits into from
Apr 25, 2024

Commits on Apr 25, 2024

  1. Split cvat/apps/iam/permissions.py into per-app permissions modules 

    This improves the dependency graph of apps with regards to IAM and other
apps. IAM is a generic app, so it should not depend on specific apps, like
`webhooks`.

It's also easier to work with a bunch of smaller files rather than one giant
one.
    @SpecLad
    SpecLad committed Apr 25, 2024
    Configuration menu
    Copy the full SHA
    ec9fb76 View commit details
    Browse the repository at this point in the history

  2. Ensure that permissions modules are always loaded 

    Failing to load one would be disastrous, so load them all in each app's
`ready()` method.
    @SpecLad
    SpecLad committed Apr 25, 2024
    Configuration menu
    Copy the full SHA
    2503653 View commit details
    Browse the repository at this point in the history

  3. Apply black and isort

    @SpecLad
    SpecLad committed Apr 25, 2024
    Configuration menu
    Copy the full SHA
    a13159c View commit details
    Browse the repository at this point in the history

  4. Update copyright headers

    @SpecLad
    SpecLad committed Apr 25, 2024
    Configuration menu
    Copy the full SHA
    807ce51 View commit details
    Browse the repository at this point in the history