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 - Persistence of shared business partners (Sites + Logistic Addresses + Legal Entities) #95

Merged

Conversation

alexsilva-CGI
Copy link
Contributor

@alexsilva-CGI alexsilva-CGI commented Apr 6, 2023

When a sharing member shares a business partner via the Gate API, the information can be persisted in the database. If the information is already present in the database, then the entries are overwritten and updated. First implementation for the Sites logic. Relationships between Legal Entities and Addresses already on place.

@alexsilva-CGI alexsilva-CGI force-pushed the feat/common-entity-model-site branch from c05c0e9 to 2aa45a8 Compare April 6, 2023 14:01
@alexsilva-CGI alexsilva-CGI marked this pull request as ready for review April 13, 2023 09:01
@alexsilva-CGI alexsilva-CGI force-pushed the feat/common-entity-model-site branch 2 times, most recently from 274ed15 to 7d45669 Compare May 4, 2023 09:19
@alexsilva-CGI alexsilva-CGI marked this pull request as draft May 4, 2023 10:09
@cezaralexandremorais cezaralexandremorais force-pushed the feat/common-entity-model-site branch from 265cfff to 32f6aab Compare May 4, 2023 17:08
@alexsilva-CGI alexsilva-CGI changed the title Feat - Persistence of shared business partners (Site) Feat - Persistence of shared business partners (Sites + Logistic Addresses + Legal Entities) May 9, 2023
@alexsilva-CGI alexsilva-CGI force-pushed the feat/common-entity-model-site branch from a471f44 to bb153b6 Compare May 9, 2023 13:46
@alexsilva-CGI alexsilva-CGI marked this pull request as ready for review May 12, 2023 09:38
Copy link
Contributor

@nicoprow nicoprow left a comment

Choose a reason for hiding this comment

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

Some refactoring needed but the functionality works and looks good to me

@nicoprow
Copy link
Contributor

updates issue #150

@alexsilva-CGI alexsilva-CGI force-pushed the feat/common-entity-model-site branch 2 times, most recently from 3393e1b to 6c0c743 Compare May 23, 2023 11:56
@alexsilva-CGI alexsilva-CGI force-pushed the feat/common-entity-model-site branch from 6c0c743 to a4f6b69 Compare May 23, 2023 12:23
@nicoprow nicoprow merged commit 9097895 into eclipse-tractusx:main May 23, 2023
@nicoprow nicoprow deleted the feat/common-entity-model-site branch May 23, 2023 12:51
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