Skip to content

[MDS-6632] - Link Orgbook Permissions#3712

Merged
matbusby-fw merged 5 commits intodevelopfrom
mds-6632-orgbook-link-permissions
Oct 20, 2025
Merged

[MDS-6632] - Link Orgbook Permissions#3712
matbusby-fw merged 5 commits intodevelopfrom
mds-6632-orgbook-link-permissions

Conversation

@matbusby-fw
Copy link
Collaborator

@matbusby-fw matbusby-fw commented Oct 20, 2025

  • Created a new role 'core_manage_orgbook' and updated the permissions
  • Adjusted the display of the associate orgbook organization section of the party form to show for anyone with this new role
  • Updated the backend to use this new role
  • Currently this role has been assigned to the c_mds_users role.
  • Role has been created on all three environments

Objective

MDS-6632

…se TypeScript and functional component structure.

- Removed `connect` and migrated to Redux hooks (`useAppSelector`) for state management.
- Introduced role-based rendering for OrgBook features using `role_manage_orgbook`.
- Updated tests to align with component refactoring.
…ess for OrgBook features across API and web components.
… type safety, integrate `role_manage_orgbook`, and update tests.
@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed for 'bcgov-sonarcloud_mds_minespace-web'

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

@sonarqubecloud
Copy link

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed for 'bcgov-sonarcloud_mds_core-web'

Failed conditions
67.6% Coverage on New Code (required ≥ 80%)
30.5% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@sonarqubecloud
Copy link

@matbusby-fw matbusby-fw added the 👍 Ready for review Pull request has been double checked by the author and is ready for comments and feedback. label Oct 20, 2025
@matbusby-fw matbusby-fw merged commit c59f9c2 into develop Oct 20, 2025
19 of 22 checks passed
@matbusby-fw matbusby-fw deleted the mds-6632-orgbook-link-permissions branch October 20, 2025 21:07
matbusby-fw added a commit that referenced this pull request Nov 5, 2025
* - Refactored `EditPartyModal` and `EditFullPartyForm` components to use TypeScript and functional component structure.
- Removed `connect` and migrated to Redux hooks (`useAppSelector`) for state management.
- Introduced role-based rendering for OrgBook features using `role_manage_orgbook`.
- Updated tests to align with component refactoring.

* Added `role_manage_orgbook` permissions and integrated role-based access for OrgBook features across API and web components.

* Refactor `EditPartyModal` and `EditFullPartyForm` components, enhance type safety, integrate `role_manage_orgbook`, and update tests.

* Update `test_expected_auth.py` to adjust `MANAGE_ORGBOOK` permissions for `PartyResource` operations.

* remove incorrectly added orgbook permission check within party_resource.py

---------

Co-authored-by: mat <matbusby@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

👍 Ready for review Pull request has been double checked by the author and is ready for comments and feedback.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants