Skip to content

fix: simplify model with types and simplify api #15

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

Merged
merged 1 commit into from
Nov 25, 2021

Conversation

millotp
Copy link
Collaborator

@millotp millotp commented Nov 24, 2021

Remove the default ObjectSerializer to use the classic one from the existing client, that mean a lot of code become useless.

  • Convert all models from class to type
  • Remove all auth other than ApiKey
  • Create common method sendRequest to handle auth

@millotp millotp requested review from damcou and shortcuts November 24, 2021 17:07
@millotp millotp self-assigned this Nov 24, 2021
@shortcuts
Copy link
Member

(reviewed in sync with @damcou and @millotp)

@millotp millotp merged commit 4423c9e into fix/echo-requester Nov 25, 2021
@millotp millotp deleted the fix/simplify-model branch November 25, 2021 16:24
millotp added a commit that referenced this pull request Nov 25, 2021
* fix: simplify model with types and simplify api (#15)
shortcuts pushed a commit that referenced this pull request Apr 22, 2022
* fix: simplify model with types and simplify api (#15)
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