-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add ExpandoObjectConverter for JavaScript object conversion #5896
Conversation
Introduced `ConverterHelper` and `ExpandoObjectConverter` to handle conversions from .NET dynamic objects to JavaScript objects. Updated integration tests to validate the conversion and sorting functionalities. Fixes #5895
|
||
internal static class ConverterHelper | ||
{ | ||
public static ObjectInstance ConvertToJsObject(Engine engine, IDictionary<string, object?> expando) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we make this internal as well?
|
||
internal static class ConverterHelper | ||
{ | ||
public static ObjectInstance ConvertToJsObject(Engine engine, IDictionary<string, object?> expando) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This method is missing a summary
|
||
internal class ExpandoObjectConverter : IObjectConverter | ||
{ | ||
public bool TryConvert(Engine engine, object value, out JsValue result) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we make this internal as well?
|
||
internal class ExpandoObjectConverter : IObjectConverter | ||
{ | ||
public bool TryConvert(Engine engine, object value, out JsValue result) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This method is missing a summary
Introduced
ConverterHelper
andExpandoObjectConverter
to handle conversions from .NET dynamic objects to JavaScript objects. Updated integration tests to validate the conversion and sorting functionalities.Fixes #5895
This change is