Skip to content
This repository has been archived by the owner on Jul 15, 2022. It is now read-only.

Cosmos JS #1763

Merged
merged 70 commits into from Mar 1, 2022
Merged

Cosmos JS #1763

merged 70 commits into from Mar 1, 2022

Conversation

ghost
Copy link

@ghost ghost commented Mar 1, 2022

Context (issues, jira)

Description / Usage

Expectations

  • Test coverage: The changes of this PR are covered by test. Unit test were added with mocks when depending on a backend/device.
  • No impact: The changes of this PR have ZERO impact on the userland. Meaning, we can use these changes without modifying LLD/LLM at all. It will be a "noop" and the maintainers will be able to bump it without changing anything.

handle payload atomic construction
fix public key when account is derivate
restruct operation builder
simulate now return int
prepareTransaction use patch format
more accuracy int value
small refactor
revert back test trace for bot
@ghost ghost self-requested a review as a code owner March 1, 2022 08:12
@vercel
Copy link

vercel bot commented Mar 1, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/ledgerhq/ledger-live-common/GWF817M91829A47XvUKAfti8gaY1
✅ Preview: https://ledger-live-common-git-fork-wa-aal-cosmos-js-ledgerhq.vercel.app

@gre gre merged commit 5978c5a into LedgerHQ:cosmos-js Mar 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants