Skip to content

Conversation

marco-figment
Copy link
Contributor

@marco-figment marco-figment commented Aug 9, 2022

Co-Authored-By: JesseKuntz [email protected]
Co-Authored-By: Trent Rogers [email protected]

📝 Description

This PR adds support for sending, receiving and staking funds in the Ledger Live Mobile app.

❓ Context

✅ Checklist

  • ℹ️ Test coverage Manual testing was performed E2E for all app flows.
  • Atomic delivery
  • No breaking changes

📸 Demo

Sync & Receive

Sync.Receive.mp4

Send

Send.mp4

Send Max

Send.Max.mp4

Delegate

Delegate.mp4

Claim rewards

Claim.Rewards.mp4

Claim rewards & Compound

Claim.Rewards.and.Compound.mp4

Redelegate

Redelegate.mp4

Undelegate

Undelegate.mp4

Operation Details

Operation.Details.mp4

marco-figment and others added 2 commits August 9, 2022 10:00
@changeset-bot
Copy link

changeset-bot bot commented Aug 9, 2022

🦋 Changeset detected

Latest commit: f5d6bcb

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
live-mobile Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added mobile Has changes in LLM translations Translation files have been touched fork Pull request base branch comes from a fork. labels Aug 9, 2022
@codecov
Copy link

codecov bot commented Aug 9, 2022

Codecov Report

Merging #890 (a56c6e6) into feat/osmosis (cf3fcaf) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

❗ Current head a56c6e6 differs from pull request most recent head f5d6bcb. Consider uploading reports for the commit f5d6bcb to get more accurate results

@@               Coverage Diff                @@
##           feat/osmosis     #890      +/-   ##
================================================
- Coverage         47.39%   47.36%   -0.04%     
================================================
  Files               638      637       -1     
  Lines             28610    28598      -12     
  Branches           7376     7375       -1     
================================================
- Hits              13560    13545      -15     
+ Misses            13882    13878       -4     
- Partials           1168     1175       +7     
Flag Coverage Δ
test 47.36% <0.00%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ve-common/src/families/osmosis/js-signOperation.ts 17.91% <0.00%> (ø)
...s/ledger-live-common/src/families/cardano/logic.ts 20.12% <0.00%> (-5.04%) ⬇️
libs/ledger-live-common/src/hw/getDeviceInfo.ts 82.22% <0.00%> (-0.76%) ⬇️
libs/ledgerjs/packages/cryptoassets/src/tokens.ts 75.00% <0.00%> (ø)
...on/src/families/stellar/js-getTransactionStatus.ts 9.09% <0.00%> (ø)
libs/ledger-live-common/src/hw/isDevFirmware.ts
...ive-common/src/families/filecoin/bridge/account.ts 21.55% <0.00%> (+0.89%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@vercel
Copy link

vercel bot commented Aug 9, 2022

@marco-figment is attempting to deploy a commit to the LedgerHQ Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Aug 9, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
live-common-tools ✅ Ready (Inspect) Visit Preview Aug 12, 2022 at 1:09PM (UTC)
native-ui-storybook ✅ Ready (Inspect) Visit Preview Aug 12, 2022 at 1:09PM (UTC)
react-ui-storybook ✅ Ready (Inspect) Visit Preview Aug 12, 2022 at 1:09PM (UTC)

@github-actions github-actions bot added the common Has changes in live-common label Aug 9, 2022
@ghost ghost self-requested a review August 10, 2022 07:13
@marco-figment marco-figment marked this pull request as ready for review August 10, 2022 18:13
…-llm/edit-memo-field

feat: refactor CosmosEditMemo and add it in Osmosis LLM
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
common Has changes in live-common fork Pull request base branch comes from a fork. mobile Has changes in LLM translations Translation files have been touched
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants