Skip to content
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

Feat/register parttype at edc #296

Conversation

eschrewe
Copy link
Contributor

  • added parttype enpoint for delivering parttype information to customer partners
  • added routine to fetch a supplier partner's CX-Id in the MaterialPartnerRelationService
  • simplified env-variables for the various api-endpoints in the backend
  • solves Register PartTypeInformation Submodel at EDC #188

Please ensure to do as many of the following checks as possible, before asking for committer review:

Copy link
Contributor

@tom-rm-meyer-ISST tom-rm-meyer-ISST left a comment

Choose a reason for hiding this comment

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

Overall great contribution! Please adjust the EDC Asset definition and check regarding my question of future combination with shellDescriptor and asset bundling.

Copy link
Contributor

@tom-rm-meyer-ISST tom-rm-meyer-ISST left a comment

Choose a reason for hiding this comment

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

Great update! I faulty desribed the semantic id with @id which should be a nested object same as dct:type. All findings are rather small :)

@Autowired
private PartTypeInformationSammMapper sammMapper;

@Operation(description = "Ednpoint that delivers PartTypeInformation of own products to customer partners. " +
Copy link
Contributor

Choose a reason for hiding this comment

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

Small typo Ednpoint -> Endpoint

Copy link
Contributor

@tom-rm-meyer-ISST tom-rm-meyer-ISST left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks a lot!

@tom-rm-meyer-ISST tom-rm-meyer-ISST merged commit ed2cbda into eclipse-tractusx:main Mar 22, 2024
13 checks passed
@tom-rm-meyer-ISST tom-rm-meyer-ISST deleted the feat/register_parttype_at_edc branch March 22, 2024 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants