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

PSP-7414 Tech debt: Remove obsolete roles from PSP frontend #3718

Merged
merged 6 commits into from
Jan 17, 2024

Conversation

asanchezr
Copy link
Collaborator

No description provided.

@asanchezr asanchezr added the tech-debt Removing technical debt label Jan 17, 2024
@asanchezr asanchezr self-assigned this Jan 17, 2024
Copy link
Contributor

✅ No secrets were detected in the code.

Copy link

codecov bot commented Jan 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (0ed025d) 74.91% compared to head (7c8de37) 73.84%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #3718      +/-   ##
==========================================
- Coverage   74.91%   73.84%   -1.07%     
==========================================
  Files        1439      947     -492     
  Lines       37184    20370   -16814     
  Branches     6973     5864    -1109     
==========================================
- Hits        27855    15042   -12813     
+ Misses       9051     5328    -3723     
+ Partials      278        0     -278     
Flag Coverage Δ
unittests 73.84% <100.00%> (-1.07%) ⬇️

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

Files Coverage Δ
source/frontend/src/features/account/Login.tsx 93.75% <100.00%> (+0.64%) ⬆️
source/frontend/src/hooks/useKeycloakWrapper.tsx 83.67% <ø> (-0.33%) ⬇️

... and 492 files with indirect coverage changes

Comment on lines 38 to 42
} else if (
keyCloakWrapper.roles?.length === 1 &&
keyCloakWrapper.hasRole(Roles.LEASE_FUNCTIONAL)
) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

this looked like a bug waiting to be found

Copy link
Contributor

✅ No secrets were detected in the code.

@asanchezr asanchezr merged commit e8cd4f2 into bcgov:dev Jan 17, 2024
8 of 9 checks passed
@asanchezr asanchezr deleted the psp-7414-tech-debt branch January 31, 2024 03:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech-debt Removing technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants