search

contentful / contentful.net

.NET Library for Contentful's Content Delivery and Management API
MIT License
95 stars 53 forks source link

Bug/asset hyperlink #345

Closed kheurterincentro closed 1 month ago

kheurterincentro commented 1 month ago

An AssetHyperlink class did exist, but wasn't used in ContentJsonConverter. This would be technically a breaking change. The reason why we would prefer this way is to avoid having to deal with strange patternmatchings on AssetStructure while the expected type should actually be AssetHyperlink. Added a new renderer for good measure (we don't actually use this renderer, but the tests failed, so it felt appropiate) @Roblinde

Roblinde commented 1 month ago

@kheurterincentro thank you for this, I'll have a look.

Roblinde commented 1 month ago

@kheurterincentro I have created a superceding PR that will include this change. It will soon be released in version 8.0.1.