Skip to content
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

Let /auth/accounts/<address> client support 0.37-0.39 format #314

Merged
merged 5 commits into from
Jul 28, 2020

Conversation

webmaster128
Copy link
Member

Based on #313.

This allows us to use one LCD API client for Comsos SDK 0.37-0.39.

Closes #302
Closes #312

Copy link
Contributor

@ethanfrey ethanfrey 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 and nice flexible solution

packages/sdk38/src/lcdapi/auth.ts Show resolved Hide resolved
Copy link
Contributor

@willclarktech willclarktech left a comment

Choose a reason for hiding this comment

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

Just a couple of typos

/**
* A Cosmos SDK base account.
*
* This type described the base account representation as returned
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo: "describes"?

* The public key of the account. This is not available on-chain as long as the account
* did not send a transaction.
*
* This was a type/value object in Cosmos SDK 0.37, changed to bach32 in Cosmos SDK 0.38 ([1])
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo: "bech32"

import { PubKey } from "../types";

/**
* Convers an integer expressed as number or string to a number.
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo: "Converts"

}

/**
* Convers an integer expressed as number or string to a number.
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo: "Converts"

@webmaster128 webmaster128 merged commit a475bce into master Jul 28, 2020
@webmaster128 webmaster128 deleted the uint64ToNumber-and-String branch July 28, 2020 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants