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 - Business Partner Output Endpoints #231

Merged

Conversation

alexsilva-CGI
Copy link
Contributor

@alexsilva-CGI alexsilva-CGI commented Jun 2, 2023

Alteration of the "Search Business Partner Output" endpoints using the persistence database instead of the SaaS, and creation of PUT endpoints for each LSA type to persist the Output data

Issues related: #218 and #220

@alexsilva-CGI alexsilva-CGI force-pushed the feat/pesistence_output_bp branch from 52c9e41 to 66f1be2 Compare June 5, 2023 13:02
@alexsilva-CGI alexsilva-CGI marked this pull request as ready for review June 6, 2023 10:35
…etch, added composite key on addresses (externalid + datatype). Altered database queries according new datatype
…composite key on Sites/LegalEntity (externalid + datatype). Changed Repository calls accounting to dataType field
…. Show 400 if there isn't an Equal Input persisted
…xed Output unit tests according to new logic, Fixed API descriptions
…ut to output data class, fixed persistence accordingly
@alexsilva-CGI alexsilva-CGI force-pushed the feat/pesistence_output_bp branch from a1ab800 to cda2f5e Compare June 7, 2023 13:00
- remove BPNs to be nullable in the output DTOs
- add legal address to legal entity output response
- unwrap legal entity data in the legal entity request DTOs
- ignore admin area level 1 in test assertions for now
@nicoprow nicoprow force-pushed the feat/pesistence_output_bp branch from cb2d3fa to d4704e5 Compare June 9, 2023 12:13
@nicoprow nicoprow merged commit 58e638e into eclipse-tractusx:main Jun 9, 2023
@nicoprow nicoprow deleted the feat/pesistence_output_bp branch June 9, 2023 12:23
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