-
Notifications
You must be signed in to change notification settings - Fork 2k
Add changeSummary API endpoint and UI components #26533
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
Merged
Merged
Changes from 13 commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
12383a7
Add changeSummary API endpoint and UI components for description sour…
harshach f2b7b66
Fix checkstyle
harshach b77ce50
Integrate DescriptionSourceBadge into UI and address PR review comments
harshach 3392ba8
Address PR review feedback: fix column FQN parsing, badge labels, bac…
harshach 145d0b7
Improve change summary description layout
harshach a4aaabc
Expand change summary support across assets
harshach 2511acc
Merge remote-tracking branch 'origin/main' into feature/change-summar…
pmbrull 7db09ab
Fix changeSummary race condition and LLMModel entity type mismatch
pmbrull 857a8ee
Fix Playwright strict mode violation and sync i18n translations
pmbrull 2c4591e
fix
pmbrull 71de2cb
Merge remote-tracking branch 'origin/main' into feature/change-summar…
pmbrull 17a13a3
fix
pmbrull 8d5e201
fix
pmbrull 0f1f4ce
Merge branch 'main' into feature/change-summary-api
pmbrull 1c4a1ea
implemented the new UI changes for AI description
Rohit0301 892495d
fixed the lint issues
Rohit0301 a18bfcb
addressed gitar comment
Rohit0301 48181a7
fixed the translations
Rohit0301 fd88967
fixed unit test
Rohit0301 b49dcdb
Merge branch 'main' into feature/change-summary-api
Rohit0301 bec52d6
addressed PR comment
Rohit0301 2da3ae0
fixed odcs playwright test
Rohit0301 532033e
Merge branch 'main' into feature/change-summary-api
harshach File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The assertions only check for presence of
changeSummary/totalEntries, which will still pass when the API returns an empty map. To validate behavior, assert thatchangeSummarycontains the updated field (e.g.,description) and thattotalEntriesis > 0 after the PATCH.