-
Notifications
You must be signed in to change notification settings - Fork 70
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
JEAN BAPTISTE ZIADE MetadataAddressForAllPayoutPrice #3225
base: master
Are you sure you want to change the base?
Conversation
Harmonization : ensure all Price components in Payout have [metadata address], was missing mainly for returnTerms->vol/var/correl and for optionPayout
✅ Deploy Preview for finos-cdm ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Background :
Target :
|
as an info, had already raised the Issue about it last year... |
@Oblongs thas it the PR mentioning today in CRWG call kidnly review if this one might be merged in coming month with Dev 6 Thanks a lot |
Can you please add a release note and reach out to a CDM Maintainer to review and approve. If this is done, we can release to CDM 6-dev on Friday, |
Release NoteBackground Purpose is to ensure consistent usage accross the model of referencing accross the model for Rationale :
What is being released ? 1 -
2 - Rationale : prior type number' is considered being both :
|
@Oblongs @lolabeis @hugohills-regnosys |
No description provided.