Skip to content

HMS-10989,HMS-10995: Remove dedupe workaround for package versions - #1117

Merged
rverdile merged 1 commit into
content-services:mainfrom
rverdile:maven-versions-fix
Jul 23, 2026
Merged

HMS-10989,HMS-10995: Remove dedupe workaround for package versions#1117
rverdile merged 1 commit into
content-services:mainfrom
rverdile:maven-versions-fix

Conversation

@rverdile

Copy link
Copy Markdown
Contributor

Summary

The API was updated to remove the duplicated versions, so the handling of the response has been updated to match. It also removes the detail's page partial dependency on the packageListSearchQuery, as the details page uses the packageVersionsListQuery instead.

Testing steps

  1. Use the backend PR: HMS-10989: Update Tang for version duplication fix content-sources-backend#1593
  2. Click around the UI and make sure all instances of package version and release are formatted as expected. Compare against stage for reference.

@rverdile
rverdile requested a review from a team as a code owner July 20, 2026 17:54
@rverdile rverdile changed the title HMS-10989: Remove dedupe workaround for Lightwell package versions HMS-10989,HMS-10995: Remove dedupe workaround for Lightwell package versions Jul 20, 2026

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@xbhouse

xbhouse commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

@xbhouse

xbhouse commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

@rverdile rverdile changed the title HMS-10989,HMS-10995: Remove dedupe workaround for Lightwell package versions HMS-10989,HMS-10995: Remove dedupe workaround for package versions Jul 20, 2026
@rverdile
rverdile force-pushed the maven-versions-fix branch from b41783c to bdfe997 Compare July 20, 2026 18:03
@codecov

codecov Bot commented Jul 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.89%. Comparing base (29cdc05) to head (bdfe997).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1117      +/-   ##
==========================================
- Coverage   64.04%   63.89%   -0.15%     
==========================================
  Files         138      138              
  Lines        5337     5337              
  Branches     1041     1041              
==========================================
- Hits         3418     3410       -8     
- Misses       1636     1645       +9     
+ Partials      283      282       -1     
Flag Coverage Δ
e2e 63.89% <ø> (-0.15%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

TenSt
TenSt previously approved these changes Jul 22, 2026

@TenSt TenSt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm!

@TenSt TenSt self-assigned this Jul 22, 2026
…sions

The API was updated to remove the duplicated versions, so the handling
of the response has been updated to match. It also removes the detail's
page partial dependency on the packageListSearchQuery, as the details page uses
the packageVersionsListQuery instead.
@rverdile

Copy link
Copy Markdown
Contributor Author

@TenSt had to rebase

@rverdile
rverdile merged commit dd96617 into content-services:main Jul 23, 2026
20 checks passed
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.

3 participants