-
Notifications
You must be signed in to change notification settings - Fork 90
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
Replace/remove PropertyMappingService #8
Comments
Hi Michael, I'd actually prefer that :-) That said, I remember looking into how I could use AutoMapper for this and didn't find an intended way of achieving it... therefore I resorted to writing something myself. But if you can get it to work, by all means: use (and share ;-)) it! :) KR, |
Hi @michiproep can you please check this on your end if it works? :) |
Hi @Khungersumit , not 100% sure what you mean? Are you talking about this issue or the #9 ? |
It is very hard to get a reply from @KevinDockx , He usually takes too much time to respond to the users who have subscribed to his courses 😏 |
Hi @michiproep You can use your existing class, I gave my custom naming here. Just add a new property in this XML request and the same property in your requestDto and see the Binding of new property as I expained with the screenshot in #9 (comment) |
That's because I'm busy creating new versions of courses asked for by users ;-) But anyway, notifying me like this or on Twitter tends to do the trick - I had a look, see original post. |
Hi @KevinDockx ,
I'm just curious if you see a problem to remove or change the PropertyMappingService since we got the same information in AutoMapper.
May-be, there is a way to utilize "mapper.ConfigurationProvider.Internal().GetAllTypeMaps()" instead of mapping all properties twice - once in profiles and once in PropertyMappingService.
Michael
The text was updated successfully, but these errors were encountered: