ansible / django-ansible-base

Apache License 2.0
18 stars 45 forks source link

Allow turning off role view requirement with setting #566

Closed AlanCoding closed 2 months ago

AlanCoding commented 2 months ago

See https://github.com/jctanner/galaxy_ng/pull/15

2024-08-10T03:26:22.8611631Z pulp-1 | pulp [b1a1dfe7d1b04d9ea9356d555cfc3c72]: galaxy_ng.app.signals.handlers:WARNING: Assignment to galaxy.collection_namespace_owner for <class 'galaxy_ng.app.models.namespace.Namespace'> violates a DAB role validation rule: {'permissions': ErrorDetail(string='Permissions for model namespace needs to include view, got: change_namespace, upload_to_namespace', code='invalid')}

In the plan for galaxy_ng, they will be doing the evaluations. So (for now at least) even if their roles don't "make sense", it's not our problem here, and this makes it so galaxy_ng can turn this off and make view-less roles.

sonarcloud[bot] commented 2 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud