-
Notifications
You must be signed in to change notification settings - Fork 1
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
DP-516 - SupplierInformation mapping added #537
Conversation
…ns and Answers separately
.ForMember(m => m.Telephone, o => o.MapFrom(m => m.Telephone)) | ||
.ForMember(m => m.Url, o => o.MapFrom(m => !string.IsNullOrWhiteSpace(m.Url) ? new Uri(m.Url) : default)); | ||
|
||
CreateMap<OrganisationInformation.PartyRole, OrganisationInformation.PartyRole>(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this necessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jakzal and @maciej-goaco
I can see lots of mapping done for Supplier Information , the supplier information holds the following data as well but I cannot see that is mapped. I am not sure should this be mapped as well? There is no description in the PR :)
public List Qualifications { get; set; } = [];
public List TradeAssurances { get; set; } = [];
public LegalForm? LegalForm { get; set; }
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will be part of the detail
of supplier information. Good spot 👍
Services/CO.CDP.OrganisationInformation.Persistence/OrganisationInformationContext.cs
Outdated
Show resolved
Hide resolved
We miss the name, but questions identifier is closer to its original intent than the question title.
Services/CO.CDP.DataSharing.WebApi/AutoMapper/DataSharingProfile.cs
Outdated
Show resolved
Hide resolved
… and correct type" This reverts commit 9b26ce1.
No description provided.