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

Fix Occhab for restricted permissions #2915

Merged
merged 5 commits into from
Feb 16, 2024
Merged

Conversation

jacquesfize
Copy link
Contributor

@jacquesfize jacquesfize commented Feb 15, 2024

This PR answers the issue on Occhab when a restricted user tries to modify its own station/habitats.

In addition, this PR change the create_user() to enable the creation of test user with detailed scope on one or more actions for a module (or all modules).

Link to the issue : #2909

@jacquesfize jacquesfize changed the base branch from master to develop February 15, 2024 09:49
@jacquesfize jacquesfize added this to the 2.14 milestone Feb 15, 2024
@jacquesfize jacquesfize changed the title Fix Occhab permissions Fix Occhab for restricted permissions Feb 15, 2024
Copy link

codecov bot commented Feb 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c024576) 78.43% compared to head (2dbe2ba) 78.43%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #2915   +/-   ##
========================================
  Coverage    78.43%   78.43%           
========================================
  Files           89       89           
  Lines         7214     7214           
========================================
  Hits          5658     5658           
  Misses        1556     1556           
Flag Coverage Δ
pytest 78.43% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jacquesfize jacquesfize force-pushed the fix/occhab_permissions branch from ea85b3a to 638824b Compare February 16, 2024 12:32
@jacquesfize jacquesfize merged commit 45e6c95 into develop Feb 16, 2024
9 checks passed
@jacquesfize jacquesfize deleted the fix/occhab_permissions branch February 16, 2024 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant