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(auth-admin): Hotfix: View model for delegations #16365

Merged
merged 2 commits into from
Oct 10, 2024

Conversation

GunnlaugurG
Copy link
Member

@GunnlaugurG GunnlaugurG commented Oct 10, 2024

What

Why

Specify why you need to achieve this

Screenshots / Gifs

Attach Screenshots / Gifs to help reviewers understand the scope of the pull request

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • Formatting passes locally with my changes
  • I have rebased against main before asking for a review

magnearun and others added 2 commits October 10, 2024 20:33
* refactor AccessCard component and make change to modal to view delegation in access control

* remove unused createdBy

* chore: nx format:write update dirty files

* refactor

* fix typo

* chore: nx format:write update dirty files

* fix type

* small refactor

* fix delete function

* chore: nx format:write update dirty files

* Adds delete modal

* chore: nx format:write update dirty files

* Update DelegationDeleteModal.tsx

* adds loading to create modal

* chore: nx format:write update dirty files

* small fixes and use view delegation modal in admin portal

* Update AccessCard.tsx

* chore: nx format:write update dirty files

* fix query and types

* chore: nx format:write update dirty files

---------

Co-authored-by: andes-it <[email protected]>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
…legationViewModal (#16347)

* adds nationalId to IdentityCard for createdBy in DelegationViewModal

* chore: nx format:write update dirty files

---------

Co-authored-by: andes-it <[email protected]>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
@GunnlaugurG GunnlaugurG requested review from a team as code owners October 10, 2024 20:37
@GunnlaugurG GunnlaugurG changed the title hotfix(auth-admin): Hotfix: View model for delegations fix(auth-admin): Hotfix: View model for delegations Oct 10, 2024
Copy link

codecov bot commented Oct 10, 2024

Codecov Report

Attention: Patch coverage is 14.28571% with 6 lines in your changes missing coverage. Please review.

Project coverage is 36.87%. Comparing base (04db252) to head (0092b50).
Report is 1 commits behind head on release/32.1.0.

Files with missing lines Patch % Lines
...c/lib/delegationAdmin/delegation-admin.resolver.ts 0.00% 6 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                 @@
##           release/32.1.0   #16365      +/-   ##
==================================================
- Coverage           36.87%   36.87%   -0.01%     
==================================================
  Files                6801     6801              
  Lines              140587   140594       +7     
  Branches            39973    39975       +2     
==================================================
+ Hits                51839    51841       +2     
- Misses              88748    88753       +5     
Flag Coverage Δ
air-discount-scheme-web 0.00% <ø> (ø)
api 3.37% <ø> (ø)
api-domains-auth-admin 48.48% <0.00%> (-0.29%) ⬇️
application-system-api 41.66% <ø> (ø)
application-template-api-modules 24.27% <ø> (-0.02%) ⬇️
services-auth-admin-api 52.13% <100.00%> (+<0.01%) ⬆️
services-auth-delegation-api 57.66% <100.00%> (+<0.01%) ⬆️
services-auth-ids-api 51.72% <100.00%> (+0.01%) ⬆️
services-auth-personal-representative 45.42% <100.00%> (+<0.01%) ⬆️
services-auth-personal-representative-public 41.53% <100.00%> (+<0.01%) ⬆️
services-auth-public-api 49.21% <100.00%> (+<0.01%) ⬆️
services-user-notification 47.01% <ø> (ø)
services-user-profile 62.17% <ø> (+0.07%) ⬆️
web 1.83% <ø> (ø)

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

Files with missing lines Coverage Δ
...-api-lib/src/lib/delegations/dto/delegation.dto.ts 100.00% <100.00%> (ø)
...lib/src/lib/delegations/models/delegation.model.ts 95.65% <ø> (ø)
...c/lib/delegationAdmin/delegation-admin.resolver.ts 0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 04db252...0092b50. Read the comment docs.

@datadog-island-is
Copy link

Datadog Report

All test runs 1e21d5f 🔗

15 Total Test Services: 0 Failed, 15 Passed
🔻 Test Sessions change in coverage: 5 decreased, 27 no change

Test Services
This report shows up to 10 services
Service Name Failed Known Flaky New Flaky Passed Skipped Total Time Code Coverage Change Test Service View
air-discount-scheme-web 0 0 0 2 0 7.69s 1 no change Link
api 0 0 0 4 0 2.62s 1 no change Link
api-domains-auth-admin 0 0 0 18 0 12.08s 1 no change Link
application-system-api 0 0 0 120 2 2m 54.74s 1 no change Link
application-template-api-modules 0 0 0 134 0 1m 52.17s 1 decreased (-4.16%) Link
auth-api-lib 0 0 0 20 0 33.5s 1 no change Link
services-auth-admin-api 0 0 0 128 0 3m 25.44s 1 decreased (-0.02%) Link
services-auth-delegation-api 0 0 0 256 0 2m 45.14s 1 no change Link
services-auth-ids-api 0 0 0 152 0 1m 5.66s 1 decreased (-0.01%) Link
services-auth-personal-representative 0 0 0 59 0 1m 23.23s 1 decreased (-0.02%) Link

🔻 Code Coverage Decreases vs Default Branch (5)

  • application-template-api-modules - jest 26.01% (-4.16%) - Details
  • services-auth-personal-representative - jest 44% (-0.02%) - Details
  • services-auth-public-api - jest 45.9% (-0.02%) - Details
  • services-auth-admin-api - jest 47.52% (-0.02%) - Details
  • services-auth-ids-api - jest 45.01% (-0.01%) - Details

Copy link
Member

@saevarma saevarma left a comment

Choose a reason for hiding this comment

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

🚀

@saevarma saevarma added the automerge Merge this PR as soon as all checks pass label Oct 10, 2024
@kodiakhq kodiakhq bot merged commit 24ce816 into release/32.1.0 Oct 10, 2024
83 of 86 checks passed
@kodiakhq kodiakhq bot deleted the hotfix/delegation-view-model branch October 10, 2024 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this PR as soon as all checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants