Skip to content

MODLD-1029: Analytical Entries added to Work Response DTO#502

Merged
PBobylev merged 3 commits into
masterfrom
MODLD-1029
Apr 7, 2026
Merged

MODLD-1029: Analytical Entries added to Work Response DTO#502
PBobylev merged 3 commits into
masterfrom
MODLD-1029

Conversation

@PBobylev

@PBobylev PBobylev commented Apr 3, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@PBobylev PBobylev requested review from pkjacob and zepheiryan April 3, 2026 10:33
@PBobylev PBobylev self-assigned this Apr 3, 2026
@PBobylev PBobylev added the ai label Apr 3, 2026
@PBobylev PBobylev force-pushed the MODLD-1029 branch 3 times, most recently from 16d49ca to 7588cb5 Compare April 3, 2026 13:17
@PBobylev PBobylev changed the title MODLD-1029: LightWork DTO introduced and added to WorkResponse DTO MODLD-1029: Analytical Entries added to Work Response DTO Apr 3, 2026
Comment thread src/main/resources/swagger.api/schema/resource/response/WorkResponse.json Outdated
Comment thread src/test/java/org/folio/linked/data/e2e/mappings/work/lightwork/LightWorkIT.java Outdated
Comment thread src/test/java/org/folio/linked/data/e2e/mappings/work/lightwork/LightWorkIT.java Outdated
@pkjacob

pkjacob commented Apr 5, 2026

Copy link
Copy Markdown
Contributor

Hi @PBobylev
I see that analyticalEntries is not part of the request DTO. So, we should preserve the existing edges when a work is updated right? Else, the edges will be lost after the PUT api call. right?

@PBobylev

PBobylev commented Apr 6, 2026

Copy link
Copy Markdown
Contributor Author

Hi @PBobylev I see that analyticalEntries is not part of the request DTO. So, we should preserve the existing edges when a work is updated right? Else, the edges will be lost after the PUT api call. right?

That's a good point, thank you!
Fixed.

@sonarqubecloud

sonarqubecloud Bot commented Apr 7, 2026

Copy link
Copy Markdown

@PBobylev PBobylev merged commit db0ff46 into master Apr 7, 2026
20 checks passed
@PBobylev PBobylev deleted the MODLD-1029 branch April 7, 2026 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants