canonical / identity-platform-admin-ui

Admin UI for the Canonical identity broker and identity provider solution
Other
6 stars 4 forks source link

fix: ui uses relative base paths. #332

Closed edlerd closed 3 months ago

edlerd commented 3 months ago

Done

Fixes #317 Fixes IAM-911 Fixes WD-12306

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 76.17%. Comparing base (96aca77) to head (91451d4). Report is 63 commits behind head on main.

:exclamation: Current head 91451d4 differs from pull request most recent head 85da4c0

Please upload reports for the commit 85da4c0 to get more accurate results.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #332 +/- ## ========================================== + Coverage 75.46% 76.17% +0.71% ========================================== Files 44 51 +7 Lines 3912 4298 +386 ========================================== + Hits 2952 3274 +322 - Misses 786 844 +58 - Partials 174 180 +6 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

edlerd commented 3 months ago

Adjusted to use the router property directly to keep this clean. Also added some test cases for the path calculation.