Cosmonaut icon indicating copy to clipboard operation
Cosmonaut copied to clipboard

Feature/collection mapping

Open roberino opened this issue 6 years ago • 2 comments

As per the issue I raised:

https://github.com/Elfocrash/Cosmonaut/issues/83

I've created fluent mapping methods which allow the consumer to map entities through an interface rather than decoratively through attributes. This approach gives the consumer both options. With the fluent approach, entities can be agnostic of the Cosmonaut library and the mapping concerns can be decoupled.

The attribute approach is still very valid and useful for most scenarios and will continue to work as before.

roberino avatar Jul 12 '19 11:07 roberino

Hey @rob thanks for that. It looks really good. A few comments before I do a full review.

Can you please uncomment the commented out tests? I also can't merge any breaking changes so I have to ask, are there any breaking changes on the API? If yes then will have to wait until v3 which will be out when the Cosmos DB SDK v3 is out and integrated with Cosmonaut.

There also seem to be some merge conflicts that need to be addressed.

Elfocrash avatar Jul 12 '19 14:07 Elfocrash

Ok...leave this with me. When I get a mo, I'll try to sort them out. Thanks :)

roberino avatar Jul 16 '19 13:07 roberino