Skip to content

MODLD-516: Update API to support IllustrativeContent of the Work#85

Merged
pkjacob merged 3 commits intomasterfrom
MODLD-516
Jan 6, 2025
Merged

MODLD-516: Update API to support IllustrativeContent of the Work#85
pkjacob merged 3 commits intomasterfrom
MODLD-516

Conversation

@AndreiBordak
Copy link
Copy Markdown
Contributor

No description provided.

coreMapper.addOutgoingEdges(instance, InstanceRequest.class, instanceDto.getCarrier(), CARRIER);
coreMapper.addOutgoingEdges(instance, InstanceRequest.class, instanceDto.getCopyright(), COPYRIGHT);
coreMapper.addOutgoingEdges(instance, InstanceRequest.class, instanceDto.getWorkReference(), INSTANTIATES);
coreMapper.addOutgoingEdges(instance, InstanceRequest.class, instanceDto.getIllustrations(), ILLUSTRATIONS);
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.

Illustrations is an outgoing edge of Work.
So, this change should be made in WorkMapperUnit.

Similarly, Changes made in "InstanceRequest.json" and "InstanceResponse.json" has to be moved to "WorkRequest.json" and "WorkResponse.json"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

Copy link
Copy Markdown
Contributor

@pkjacob pkjacob left a comment

Choose a reason for hiding this comment

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

Illustrations is an outgoing edge of Work.
So, change should be made in WorkMapperUnit.

Similarly, Changes made in "InstanceRequest.json" and "InstanceResponse.json" has to be moved to "WorkRequest.json" and "WorkResponse.json"

@AndreiBordak AndreiBordak requested a review from pkjacob January 3, 2025 08:13
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jan 6, 2025

@pkjacob pkjacob merged commit 11df62d into master Jan 6, 2025
@PBobylev PBobylev deleted the MODLD-516 branch March 12, 2025 14:32
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