-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
v2 with userInfo #4600
base: v2withActivity
Are you sure you want to change the base?
v2 with userInfo #4600
Conversation
APK file: https://www.kaminsky.me/nc-dev/android-artifacts/11087.apk |
IT test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/11087 |
Codacy432Lint
SpotBugs (new)
SpotBugs (master)
Android-library is not set to master branch in build.gradle |
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
* {@link String} with the display name retrieved from the server. | ||
* Call {@link RemoteOperationResult#getData()}.get(0) to get it. | ||
*/ | ||
@Override | ||
@Deprecated |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it is deprecated it should still work, so this just leads to an exception at runtime, so removing the method (leading to a compiler error) would be preferable imho.
What do you think @ezaquarii ?
a8f4dc9
to
a14318f
Compare
3223f18
to
6eb4ac7
Compare
This extends #4900 and uses new GetUserInfoRemoteOperation