-
Notifications
You must be signed in to change notification settings - Fork 5
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
fix permissions update from config using requests code path #610
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #610 +/- ##
=======================================
Coverage 80.88% 80.88%
=======================================
Files 73 73
Lines 10201 10191 -10
Branches 1825 1825
=======================================
- Hits 8251 8243 -8
+ Misses 1626 1625 -1
+ Partials 324 323 -1 ☔ View full report in Codecov by Sentry. |
Added #612 which tests the |
Tested using
magpie.register.magpie_register_permissions_from_config
from a script with a set of Permission entries and a remote Magpie instance URL. Updates applied where things broke.