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

Bumps admin UI to v1.17.0. (PP-1022) #1702

Merged
merged 1 commit into from
Mar 1, 2024

Conversation

tdilauro
Copy link
Contributor

@tdilauro tdilauro commented Feb 29, 2024

Description

Bumps admin UI to v1.17.0.

Note: This is ready for review, but holding in draft status until Admin UI v1.17.0 has been tested in the development environment.

Motivation and Context

Pick up fix for missing SAML IdPs when none have already been selected for a patron auth integration.

[Jira PP-1022]

How Has This Been Tested?

The effect of this change will be tested when the next CM release is deployed to staging.

Checklist

  • N/A - I have updated the documentation accordingly.
  • All new and existing tests passed.

@tdilauro tdilauro requested a review from a team February 29, 2024 22:17
Copy link

codecov bot commented Feb 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.21%. Comparing base (a0e01a7) to head (a8ba028).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1702   +/-   ##
=======================================
  Coverage   90.21%   90.21%           
=======================================
  Files         245      245           
  Lines       28199    28199           
  Branches     6447     6447           
=======================================
  Hits        25441    25441           
  Misses       1822     1822           
  Partials      936      936           
Flag Coverage Δ
Api 75.98% <100.00%> (ø)
Core 59.84% <100.00%> (ø)
migration 26.96% <0.00%> (ø)

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

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

Copy link
Member

@jonathangreen jonathangreen left a comment

Choose a reason for hiding this comment

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

🎉

@tdilauro tdilauro marked this pull request as ready for review March 1, 2024 13:57
@tdilauro tdilauro merged commit b96e6f4 into main Mar 1, 2024
28 checks passed
@tdilauro tdilauro deleted the feature/bump-admin-ui-to-v1.17.0 branch March 1, 2024 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants