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

Stacks integration - rebased #16

Merged
merged 953 commits into from
Apr 17, 2023

Conversation

ghost
Copy link

@ghost ghost commented Apr 13, 2023

📝 Description

Branch to address for you all changes and conflicts since the initial Stacks integration, as a courtesy for the long wait.


One last issue at the moment that I still couldn't figure out 🤔
bn.js lib not found while building live-common, even though it's added in package.json:

...
@ledgerhq/live-common:build: src/families/stacks/bridge/account.ts(5,16): error TS2307: Cannot find module 'bn.js' or its corresponding type declarations.
@ledgerhq/live-common:build: src/families/stacks/bridge/utils/utils.ts(2,16): error TS2307: Cannot find module 'bn.js' or its corresponding type declarations.
@ledgerhq/live-common:build: src/families/stacks/types.ts(2,16): error TS2307: Cannot find module 'bn.js' or its corresponding type declarations.

❓ Context

  • Impacted projects: ``
  • Linked resource(s): ``

✅ Checklist

  • Test coverage
  • Atomic delivery
  • No breaking changes

📸 Demo

🚀 Expectations to reach

Please make sure you follow these Important Steps.

Pull Requests must pass the CI and be internally validated in order to be merged.

valpinkman and others added 30 commits March 24, 2023 15:27
)

* test(LLD): install and restore set of apps flow e2e tests

* test(lld): update screenshots (macos-latest)

lld, test, screenshot

* test(lld): update screenshots (ubuntu-latest)

lld, test, screenshot

* test(lld): update screenshots (windows-latest)

lld, test, screenshot

---------

Co-authored-by: live-github-bot[bot] <105061298+live-github-bot[bot]@users.noreply.github.com>
fix: change discover v2 wording
* style(llm): sync onboarding pixel polish

* feat(vertical timeline): fix vertical margins & alignement

* feat(vertical timeline): indicator size

* feat(vertical timeline): active colors & fontweight

* feat(vertical timeline): implement dashed lines (not fully working on android)

* feat(vertical timeline): dashed lines working on android

* fix(native ui): metro config

* fix(vertical timeline): colors, alignment and small details

* feat(vertical timeline): check icon size

* fix(vertical timeline): top segment height value

* feature(native ui): new component "continue on device"

* fix(native ui): styled-components warnings about missing units

* feat(vertical timeline): expose BodyText and SubtitleText

* feat(divider): add `text` prop

* feat(vertical timeline): rework story

* feat(llm sync onboarding): ContinueOnDevice, BodyText, wording

* fix(native-ui): imports

* fix(sync-onboarding): wording

* feat(sync onboarding): ui changes in seed step

* feat(sync onboarding): remove feature toggle

* feat(nativeui verticaltimeline): auto scroll

* feat(sync-onboarding): latest changes to design

* feat(StorylyStories): play icon, blurred illustration in StoryGroupItem

fe

* changeset

* feat(SyncOnboarding): add placeholder icon for ContinueOnStax

* fix(synconboarding): wording

* fix(defaultFeatures): default apps in deviceInitialApps

* fix(ContinueOnStax): memoize component

* fix(native-ui): remove unused prop type

---------

Co-authored-by: Olivier Freyssinet <[email protected]>
…/LIVE-4282-removed-unnecessary-eq-id-reassignment
…essary-eq-id-reassignment

LLM - Bugfix/live 4282 removed unnecessary eq id reassignment
#2828)

* BACK-4796: Prefix testnet token ticker with "t" in transaction config (#2690)

BACK-4796: Prefix testnet token ticker with "t" in transaction config payload, to be consistent with CAL signature

* wip env test broken

* revert env change

* BACK-4796/BACK-5015: Prefix testnet token ticker with "t" in EVM importers (#2895)

BACK-4796 / BACK-5015: Same fix for EVM importer

* BACK-4796: generate updated signatures for testnets (#2907)

* Update env.ts

* BACK-4796: Remove ethereum.json

* BACK-4796: Update ledgerjs with latest CAL data (e97b76d6d9 2023/03/23)

* BACK-4796: Update test to match convention

* ci: update cryptoasset definition

---------

Co-authored-by: jnicoulaud-ledger <[email protected]>
Co-authored-by: Julien Nicoulaud <[email protected]>
…XPlayer

Feat/live-5401 Refactor webviews + implement PTX custom player
…XPlayer

Feat/live-5401 Refactor webviews + implement PTX custom player
…3052)

chore: add a rule that prevent 'lodash' to be directly imported

instead, you should use 'lodash/memoize' module import (or whatever function you need)
valpinkman and others added 26 commits April 7, 2023 11:49
* fix: correctly trigger software checks during sync onboarding on LLM

* fix: correctly trigger software checks during sync onboarding on LLD
* feat(navigation): poc for navigating to whitelisted live apps from the onboarding


chore: revert changes of POC


chore: lint


fix bad importt


fix protect example link


feat(welcome): recover sign in link logic


remove test URL


chore: add todos


revert(llm): recover sign-in unfinished work

* refactor(terms): remove useless useTermsAccept & fix type

* fix(DeeplinksProvider): linting, typing

* chore(llm): changeset

* fix(deeplink): use existing navigators for non-onboarded deeplinking config

* revert(DeeplinksProvider): useless deeplink

* revert(BleDevicePairingFlow): revert the hacky fix for deeplink handling

* refactor(terms): small polish

* style(BleDevicePairingFlow): lint (unused imports)

* fix(llm terms): memoize value passed to context provider

* ci(Fastfile): fix syntax error
* feat: add stake flow to asset page

* fix: fix the types

* chore: add changeset

* chore: address pr comments
* feat(mobile): add Stax enabled NFT in the account mock generator

feat(mobile): display Stax enabled NFTs tag in gallery

feat(mobile): add designed for stax drawer in NFT view

feat(mobile): disable lockscreen edit when nft is stax enabled

feat(mobile): add logic to stax enabled nft display

refactor(mobile): code cleanup in nft fixtures

refactor(mobile): pixel polish lockscreen flow

chore: generate changesets

fix(types-live): update documentation to reflect nft metadata changes

feat(mobile): add analytics related to stax enabled NFTs

fix(DesignedForStax): colors


fix(NftMedia): pass Stax NFT metadata banner as children


fix(mobile): code cleanup and pixel polish for stax enabled nfts

feat(mobile): add a debug setting to switch to the staging nft metadata provider

feat(desktop): add debug setting to switch to the staging nft metadata service

refactor(mobile): pixel polish stax enabled nfts

* ci(Fastfile): fix syntax error

---------

Co-authored-by: Olivier Freyssinet <[email protected]>
* feat: modified dummy app server and started adding tests

* feat: PTX app now just a simple HTML app that shows the query params used

* feat: Buy test initial assert working

* feat: added further buy tests

* feat: added lang and theme test
* test(detox): fix deeplink tests

* fix(e2e bridge client): import the correct acceptTerms function

* style(llm terms): lint

* refactor(terms): reorganise, document and rename terms logic

* style(terms): rename variable

* refactor(terms): move data from bare AsyncStorage to settings

* style(terms): rename
…edger (#3165)

* feat: use backend supportUrl for apps if available
* chore: align support url fallback for apps llm/lld
Various performance and documentation improvements :
- Updated documentation regarding fonts loading to prevent fonts flickering
- Updated package.json to fix path issues on certain consumers
- Updated minimal working examples (NextJS)
- Removed Icon component from Button component to avoid embarking all icons when using buttonVarious performance and documentation improvements :
- Updated documentation regarding fonts loading to prevent fonts flickering
- Updated package.json to fix path issues on certain consumers
- Updated minimal working examples (NextJS)
- Removed Icon component from Button component to avoid embarking all icons when using button
Deleted all un-used icons weights (Only Medium icons remains)
Replaced legacy icons with medium icons in LLD and LLM
updated and rebased branch on develop, fixed test issues and modifying changes made on develop.
@ghost ghost force-pushed the feat/stacks-rebased branch 2 times, most recently from 9956a50 to 48cfaf3 Compare April 14, 2023 07:34
@emmanuelm41 emmanuelm41 marked this pull request as ready for review April 17, 2023 14:36
@emmanuelm41 emmanuelm41 merged commit 778f5bc into Zondax:feat/stacks Apr 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.