Skip to content
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

Implement kiota_serialization_text #22

Merged
merged 9 commits into from
Apr 9, 2024
Merged

Conversation

ricardoboss
Copy link
Contributor

Fixes #21

Enum deserialisation is currently not possible, because Dart doesn't have reflection. I linked to an issue that proposes to add abstract static members to classes, which would in turn allow the Dart SDK to provide static access to the values member of enums which would allow us to use it to deserialise a specific value.

In the meantime we either don't support enum deserialisation (not preferred) or we implement a workaround which would increase the amount of generated code (via static factories that register each available enum).

@ricardoboss ricardoboss requested a review from andrueastman April 1, 2024 12:38
@ricardoboss ricardoboss force-pushed the issues/21-text-serialization branch from db1ae70 to f6331f7 Compare April 7, 2024 13:59
@ricardoboss ricardoboss force-pushed the issues/21-text-serialization branch from f6331f7 to 6b83704 Compare April 9, 2024 23:15
@ricardoboss
Copy link
Contributor Author

Just rebased (no conflicts); continuing with merge

@ricardoboss ricardoboss merged commit 8a4dff5 into main Apr 9, 2024
3 checks passed
@ricardoboss ricardoboss deleted the issues/21-text-serialization branch April 9, 2024 23:16
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.

kiota_serialization_text: Text serialization
2 participants