Skip to content
This repository was archived by the owner on Dec 20, 2025. It is now read-only.

fix(retrofit2): fix retrofit2 issues (backport #1875)#1876

Merged
mergify[bot] merged 1 commit intorelease-1.37.xfrom
mergify/bp/release-1.37.x/pr-1875
Feb 27, 2025
Merged

fix(retrofit2): fix retrofit2 issues (backport #1875)#1876
mergify[bot] merged 1 commit intorelease-1.37.xfrom
mergify/bp/release-1.37.x/pr-1875

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Feb 27, 2025

Addressed the following two issues which occurred due to restrictions in retrofit2 :

* test(retrofit2): add test to demonstrate the following error when QueryMap is used without generics:

`java.lang.IllegalArgumentException: Map must include generic types (e.g., Map<String, String>)`

* fix(retrofit2): fix non-generics QueryMap which causes the following error:

`java.lang.IllegalArgumentException: Map must include generic types (e.g., Map<String, String>)`

* fix(retrofit2): fix `A @path parameter must not come after a @Query` error

(cherry picked from commit bdf8922)
@dbyron-sf dbyron-sf added the ready to merge Approved and ready for merge label Feb 27, 2025
@mergify mergify bot added the auto merged label Feb 27, 2025
@mergify mergify bot merged commit 0da41f4 into release-1.37.x Feb 27, 2025
4 checks passed
@dbyron-sf dbyron-sf deleted the mergify/bp/release-1.37.x/pr-1875 branch February 27, 2025 23:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants