Skip to content

Conversation

@kuhnroyal
Copy link

Allow sync (de)serialization & fix some parameter bugs

  • serialize path/query/form parameters synchronously
  • force cast non-nullable parameters

Fix built_value collection builders in serializers

* serialize path/query/form parameters synchronously
* force cast non-nullable parameters
@kuhnroyal kuhnroyal self-assigned this Sep 1, 2023
@ahmednfwela
Copy link
Member

@kuhnroyal
my plan for the repository was to have 2 versions
ISyncRepositoryBase, IAsyncRepositoryBase, where all implementations are required to provide the ISyncRepositoryBase version.
and the IAsyncRepositoryBase can fall back to calling the ISyncRepositoryBase.

and when sending requests/receiving responses, we use the IAsyncRepositoryBase

@ahmednfwela
Copy link
Member

the plan right now is:

  1. finalize the repository base classes
  2. make json_serializable work
  3. add tests

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.

2 participants