Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use release v0.1.5 js-sdk #5

Merged
merged 4 commits into from
Nov 26, 2019
Merged

Use release v0.1.5 js-sdk #5

merged 4 commits into from
Nov 26, 2019

Conversation

mascot6699
Copy link
Member

  • Use release v0.1.5 which gets rid of init function call before using other functions
  • Combines all response handlers

Copy link
Member

@quixote911 quixote911 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

jsBridge.executeAsync(bridgeRequest);
}

public void putAddressMap(CruxAddressMapping newAddressMap, final CruxClientResponseHandler<CruxPutAddressMapSuccess> handler){
public void putAddressMap(HashMap newAddressMap, final CruxClientResponseHandler<CruxPutAddressMapSuccess> handler){

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we add HashMap<String, CruxAddress>?

@mascot6699 mascot6699 merged commit 63bec05 into master Nov 26, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants