Skip to content

fix: unmarshalled nested-field error - #374

Draft
damyanpetev wants to merge 1 commit into
mkirova/nullablefrom
dpetev/nullable-unmarshalled-extra-fix
Draft

fix: unmarshalled nested-field error#374
damyanpetev wants to merge 1 commit into
mkirova/nullablefrom
dpetev/nullable-unmarshalled-extra-fix

Conversation

@damyanpetev

Copy link
Copy Markdown
Member

Description

Additional fix awaiting on #373 to be merged and synced into the target branch to enable skipped tests;

Items whose nested type has public primitive-typed fields crash schema creation (JsonDataSourceSchema.Commit stores typed field getters in a Func<object, object>[]ArrayTypeMismatchException; the property arrays already use Delegate[]).

Type of Change (check all that apply):

  • Bug fix
  • New functionality
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactoring (no functional changes)
  • Documentation
  • Demos
  • CI/CD
  • Tests
  • Changelog

Component(s) / Area(s) Affected:

Combo

How Has This Been Tested?

  • Unit tests
  • Manual testing
  • Automated e2e tests

Test Configuration:

  • .NET version:
  • Hosting model:
  • Browser(s):
  • OS:

Screenshots / Recordings

Checklist:

  • All relevant tags have been applied to this PR
  • This PR includes unit tests covering all the new code
  • This PR includes API docs for newly added methods/properties
  • This PR includes general feature table updates in the root README.MD
  • This PR includes CHANGELOG.MD updates for newly added functionality
  • This PR contains breaking changes
  • This PR includes behavioral changes and the feature specification has been updated with them
  • Accessibility (ARIA, keyboard navigation, focus management) has been verified

Closes #

@damyanpetev
damyanpetev marked this pull request as draft August 28, 2026 15:57
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.

1 participant