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(services-auth-ids-api): Revert removing of raw: true for clients #10449

Merged
merged 2 commits into from Mar 11, 2023

Conversation

saevarma
Copy link
Member

What

Reverting removal of raw: true for clients.

Why

To fix regression in IDS.

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

@saevarma saevarma requested review from a team as code owners March 11, 2023 15:01
@saevarma saevarma requested review from snaerseljan and removed request for snaerseljan March 11, 2023 15:01
@saevarma saevarma added the automerge Merge this PR as soon as all checks pass label Mar 11, 2023
@codecov
Copy link

codecov bot commented Mar 11, 2023

Codecov Report

Merging #10449 (e50ca3a) into main (f9da17d) will increase coverage by 0.00%.
The diff coverage is 0.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main   #10449   +/-   ##
=======================================
  Coverage   39.69%   39.69%           
=======================================
  Files        4716     4716           
  Lines       85588    85589    +1     
  Branches    21447    21447           
=======================================
+ Hits        33977    33978    +1     
  Misses      51611    51611           
Flag Coverage Δ
api 5.08% <ø> (ø)
auth-api-lib 0.35% <0.00%> (-0.01%) ⬇️
services-auth-admin-api 45.52% <0.00%> (+0.09%) ⬆️
services-auth-delegation-api 60.07% <0.00%> (+<0.01%) ⬆️
services-auth-ids-api 55.67% <0.00%> (+<0.01%) ⬆️
services-auth-personal-representative 47.88% <0.00%> (-0.06%) ⬇️
services-auth-personal-representative-public 45.90% <0.00%> (-0.01%) ⬇️
services-auth-public-api 53.34% <0.00%> (-0.01%) ⬇️

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

Impacted Files Coverage Δ
...bs/auth-api-lib/src/lib/clients/clients.service.ts 18.88% <0.00%> (-0.11%) ⬇️

... 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 f9da17d...e50ca3a. Read the comment docs.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@kodiakhq kodiakhq bot merged commit b2b56fa into main Mar 11, 2023
36 checks passed
@kodiakhq kodiakhq bot deleted the ids-api branch March 11, 2023 16:21
saevarma added a commit that referenced this pull request Mar 13, 2023
…10449)

* Use sequelize include to load associations.

* Revert removing raw for clients.
saevarma added a commit that referenced this pull request Mar 13, 2023
…10449)

* Use sequelize include to load associations.

* Revert removing raw for clients.
kodiakhq bot added a commit that referenced this pull request Mar 13, 2023
…min api. (#10455)

* fix(auth-admin-api): Use nestjs global prefix instead of ingress rewrite. (#10415)

Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>

* fix(auth-admin-web): Fixes in auth-api-lib for auth-admin-web (#10448)

* Fix auth-api-lib resource auth-admin-web.

* Remove temporary code

---------

Co-authored-by: Eiríkur Heiðar Nilsson <eirikur@nilsson.is>

* fix(services-auth-ids-api): Revert removing of raw: true for clients (#10449)

* Use sequelize include to load associations.

* Revert removing raw for clients.

* fix(auth-admin-web): Fix get for api resources in auth admin api. (#10457)

Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>

---------

Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
Co-authored-by: Eiríkur Heiðar Nilsson <eirikur@nilsson.is>
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.

None yet

2 participants