-
Couldn't load subscription status.
- Fork 94
Georgina Tarres CDM Party Model - Legal Entity and Party Refactor #4043
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
base: master
Are you sure you want to change the base?
Georgina Tarres CDM Party Model - Legal Entity and Party Refactor #4043
Conversation
CDM Party Model - Legal Entity and Party RefactorBackground The current CDM party model has been updated to fix structural inconsistencies and missing elements. In the previous design, the This release unifies both representations by making Additionally, the model lacked support for distinguishing between natural persons acting as principals and those acting as agents of an organisation. This release strengthens the design by making this distinction clear and consistent. What is being released? Foundational Types New types have been created or refactored, conforming the base concepts used in the model:
Identifiers and Enums The model has been updated to provide a consistent approach for party and entity identifiers, which are now both represented by the following types:
Deprecated Types and Replacements The types above have been deprecated and replaced to ensure consistency:
Review Directions Changes can be reviewed in PR: #4043 Note This comment was generated via Rosetta. |
✅ Deploy Preview for finos-cdm ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
This PR is not complete yet. It currently contains the first commit with structural changes to the types related to party representation. A second commit will follow to fix and update the references in the model. Please note that the build will be broken until those updates are applied. |
|
This PR solves ISSUE #3319 |
|
What is being released? Updated references and mappings Note This comment was generated via Rosetta. |
|
@CDM-ReleaseManagement-OT @lolabeis contributed the refactor to the party types, and, in a second commit, the references as well as their mappings. We haven’t been able to test them because the build fails as party mappers are failing - probably because the structure of the types have changed and need to have the extractions updated to the new paths. The mappers that fail are the following ones: Do you want us to fix them? |
Yes, but better to wait until your changes (ex-mappings) have all been confirmed. |
@lolabeis thanks for the reply. I just wanted to check which changes you are referring to. Are there any recent or upcoming updates to old mappings that might affect us? |
|
What is being released? Update account and ancillaryEnity types Note This comment was generated via Rosetta. |
I was referring to your model changes in this PR, I adjusted my wording above to make it clear ☝️. It's best to work on mappings at the end. |
|
@gtarres I've now had a look at your PR. Before sending detailed review comments, I wanted to ask about the discrepancy between the latest spreadsheet analysis LegalEntity-Party-Proposal-updated-Sept2025.xlsx and what has been built - For example:
(There are many more instances where Party has been substituted for LegalPerson) Is there a reason for the design change? 🙏 |
|
@lolabeis thanks for taking a look. The reason of the change from Extraction from #3319 (comment) : Also, I have updated the spreadsheet shared in the last issue comment, since it wasn't the final version reflecting this update. Apologies for that. Attaching it here too: |
|
Hi @gtarres Thanks for your response. That's quite a radical take, but on reflection it's a good one because it keeps with the change's intent. Effectively it removes a lot of occurrences of the "Party" type in the model, and leaves only those where a role is being specified. One important consequence is that the corresponding attributes should also be renamed to scrub out the word "party". Please have a look at and revert on my comments on the spreadsheet (shared separately). I suggest to publish it back onto the issue once we have the final version. |
|
Hi @lolabeis thank you for the clarification. To confirm, should the attribute name updates be included in a new commit of this PR? Also, is there an estimated date for the release of the synonyms deprecation? I’m asking to prevent any potential impacts this could have on this branch and to reassess whether the mapper fixes are really necessary. Thanks in advance, |

No description provided.