-
Notifications
You must be signed in to change notification settings - Fork 159
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
Generate separate stellar-sdk-javadoc.jar release artifact #519
Generate separate stellar-sdk-javadoc.jar release artifact #519
Conversation
@@ -39,4 +39,22 @@ public static Page<AssetResponse> execute(OkHttpClient httpClient, HttpUrl uri) | |||
public Page<AssetResponse> execute() throws IOException, TooManyRequestsException { | |||
return this.execute(this.httpClient, this.buildUri()); | |||
} | |||
|
|||
@Override |
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.
looks good, can you add a test case in AssetsRequestBuilderTest
to assert these new setters result in expected output fromAssetsRequestBuilder.buildUri()
?
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.
cursor
, order
and limit
in AssetsRequestBuilder object.
I just noticed that I mistakenly pushed the code from my other branch to this branch, so now the content of this PR has changed. I have updated the PR description. 🤦 So I think we need to re-review this PR, and if there are any issues, I will submit another PR to fix it. |
fortunately, this only merged to 'soroban' , should I revert the merge? |
Before, in order to release the soroban version more conveniently, the changes related to migrating to GitHub Actions were also added to the Sorry for any inconvenience caused. |
@overcat , since this is the last commit on |
This is acceptable, can you update it in the soroban branch? thank you. |
@overcat , i pushed the change, please pull latest from |
Closes #454
Note:
IDEA users need to enable the automatic download document function.
See https://intellij-support.jetbrains.com/hc/en-us/community/posts/206834305-Automatically-download-sources-documentation-from-maven-working-great