-
Notifications
You must be signed in to change notification settings - Fork 7.3k
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
POST not work with @field annotaion #3967
Comments
Here's a test case showcasing that it can work: retrofit/retrofit/src/test/java/retrofit2/RequestFactoryTest.java Lines 2608 to 2621 in 23d0461
Please supply a failing test case if you think something is broken. |
I think this simple test may be not sufficient. |
this is the code working with OkHttpClient:
|
this is the code not working with retrofit2:
|
How is it not working? |
I can't trace much deeper. |
https://github.com/Free-Web-Movement/android-base-library this is a repository can reproduce this bug. |
Retrofit's retrofit/retrofit/src/main/java/retrofit2/RequestBuilder.java Lines 89 to 91 in 87e0c81
I need a self-contained test case to debug anything here, so I'm going to preemptively close. If you can come up with a failing test case I can take a look. |
this is the failing test case and you don't take a look. |
https://github.com/Free-Web-Movement/android-base-library |
What kind of issue is this?
Question. This issue tracker is not the place for questions. If you want to ask how to do
something, or to understand why something isn't working the way you expect it to, use Stack
Overflow. https://stackoverflow.com/questions/tagged/retrofit
Bug report. If you’ve found a bug, spend the time to write a failing test. Bugs with tests
get fixed. Here’s an example: https://gist.github.com/swankjesse/6608b4713ad80988cdc9
Feature Request. Start by telling us what problem you’re trying to solve. Often a solution
already exists! Don’t send pull requests to implement new features without first getting our
support. Sometimes we leave features out on purpose to keep the project small.
version 2.9.0
The text was updated successfully, but these errors were encountered: