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

Refactored rest client #103

Merged
merged 26 commits into from
May 24, 2024
Merged

Refactored rest client #103

merged 26 commits into from
May 24, 2024

Conversation

fsufyan
Copy link
Contributor

@fsufyan fsufyan commented Apr 26, 2024

The PR mainly aims to fix the rest client response null issues due to response fields being in a snake_case.
• Ditch chopper and use code generator with retrofit for rest client APIs.
• Add missing socket party functions: accept/remove members and close.
• Enhance functions interface by hiding internal models and using client defined models.
• Add missing client functions: importFacebookFriends and importSteamFriends.
• Add documentation comments for client functions.
• Revised tests.

@lugehorsam lugehorsam merged commit ed255e8 into heroiclabs:main May 24, 2024
1 check passed
@fsufyan fsufyan deleted the refactored-rest branch May 24, 2024 14:20
@ilmalte
Copy link
Collaborator

ilmalte commented Aug 1, 2024

Hey @lugehorsam, are you guys planning to release this changes anytime soon?

@mofirouz
Copy link
Member

mofirouz commented Aug 1, 2024

Hi @ilmalte We are looking for a maintainer of our community Dart client, if you'd like to maintain this, kindly email me with mo AT heroiclabs.com.

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.

4 participants