Skip to content

OBPIH-6160 Restrict Location type creation and editing to superusers#5587

Merged
alannadolny merged 1 commit intodevelopfrom
ft/OBPIH-6160
Nov 4, 2025
Merged

OBPIH-6160 Restrict Location type creation and editing to superusers#5587
alannadolny merged 1 commit intodevelopfrom
ft/OBPIH-6160

Conversation

@alannadolny
Copy link
Collaborator

No description provided.

@alannadolny alannadolny self-assigned this Oct 31, 2025
@github-actions github-actions bot added type: feature A new piece of functionality for the app domain: backend Changes or discussions relating to the backend server labels Oct 31, 2025
@codecov
Copy link

codecov bot commented Oct 31, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 8.49%. Comparing base (1bb7314) to head (ce40db1).
⚠️ Report is 175 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff              @@
##             develop   #5587      +/-   ##
============================================
- Coverage       9.12%   8.49%   -0.63%     
+ Complexity      1170    1114      -56     
============================================
  Files            701     711      +10     
  Lines          45281   45569     +288     
  Branches       10851   10902      +51     
============================================
- Hits            4131    3872     -259     
- Misses         40497   41123     +626     
+ Partials         653     574      -79     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

'transactionEntry' : ['edit', 'delete', 'save', 'update'],
'user' : ['impersonate'],
'productsConfigurationApi' : ['downloadCategories', 'importCategories'],
'locationType' : ['create', 'edit', 'delete'],
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how about the update, save? I don't remember exactly how those rules work, but I know that update and save are usually different controller actions than create and edit

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't find where update and save are used. I blocked only those actions that can be called from the UI to avoid blocking actions that shouldn't be blocked.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alannadolny alannadolny merged commit 3bc0033 into develop Nov 4, 2025
7 checks passed
@alannadolny alannadolny deleted the ft/OBPIH-6160 branch November 4, 2025 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: backend Changes or discussions relating to the backend server type: feature A new piece of functionality for the app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants