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.

Sign up for GitHub

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

Jump to bottom

Implement possibility to add payload for API calls #53

Open
Prominence opened this issue Oct 24, 2022 · 0 comments
Open

Implement possibility to add payload for API calls #53

Prominence opened this issue Oct 24, 2022 · 0 comments
Assignees
@Prominence
Labels
enhancement New feature or request

Comments

@Prominence
Copy link
Owner

Prominence commented Oct 24, 2022 •
edited
Loading

Redesign RequestExecutor and HttpClient to support adding custom payload with further JSON serialization.

Possible solutions:

  1. Replace String getResponse(ApiVariant variant) and String getResponse(ApiVariant variant, Method httpMethod) methods with method that receives an object that aggregates all request parameters(to get rid of method overloading).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Prominence Prominence

Labels
enhancement New feature or request
Projects
OpenWeatherMap Java API Project
Status: In Progress
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Prominence