Skip to content

Fix beta conflicts and enhance EmbeddedObject deserialization#74

Draft
reeshika-h wants to merge 32 commits into
betafrom
fix/beta-conflicts
Draft

Fix beta conflicts and enhance EmbeddedObject deserialization#74
reeshika-h wants to merge 32 commits into
betafrom
fix/beta-conflicts

Conversation

@reeshika-h

Copy link
Copy Markdown
Contributor

Resolve conflicts from the beta branch and improve the deserialization process for IEmbeddedObject by introducing EmbeddedObject and EmbeddedObjectConverter. Add unit tests to ensure functionality and preserve custom fields during deserialization. Additionally, update documentation and remove outdated references.

harshitha-cstk and others added 30 commits April 6, 2026 17:27
feat: Add Live Preview editable tags support (addEditableTags/addTags)
DX | 11-05-2026 | Release dev -> stg
…-bump

feat: version bump and changelog for Live Preview editable tags
chore: back-merge master into development
chore: back-merge master into development
…ct deserialization

IEmbeddedObject is an interface and cannot be instantiated directly by
Newtonsoft.Json. Add EmbeddedObject as a concrete implementation covering
both IEmbeddedEntry and IEmbeddedAsset, and EmbeddedObjectConverter to
redirect deserialization from the interface to the concrete class.

EmbeddedObject uses [JsonExtensionData] so custom fields on embedded
entries and assets are preserved rather than silently dropped.
…ect deserialization

Add EmbeddedObject as a concrete implementation of IEmbeddedObject covering
both IEmbeddedEntry and IEmbeddedAsset. Add EmbeddedObjectConverter to redirect
deserialization from the interface to the concrete class. Custom fields on
embedded entries and assets are preserved via [JsonExtensionData].

Add unit tests covering CanConvert isolation, known field mapping, nested
object and array fields in [JsonExtensionData], full Delivery API response
shape, and regression guard for the interface instantiation crash.
fix: add EmbeddedObject and EmbeddedObjectConverter to resolve IEmbeddedObject deserialization failure
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants