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

Feat/integrate aptos #8396

Draft
wants to merge 82 commits into
base: develop
Choose a base branch
from
Draft

Feat/integrate aptos #8396

wants to merge 82 commits into from

Conversation

hedi-edelbloute
Copy link
Member

βœ… Checklist

  • npx changeset was attached.
  • Covered by automatic tests.
  • Impact of the changes:
    • ...

πŸ“ Description

Backport of initial PR : #5964

Aptos integration feature branch

❓ Context


🧐 Checklist for the PR Reviewers

  • The code aligns with the requirements described in the linked JIRA or GitHub issue.
  • The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
  • There are no undocumented trade-offs, technical debt, or maintainability issues.
  • The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
  • Any new dependencies have been justified and documented.
  • Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

Vladyslav Belyokhin and others added 30 commits March 6, 2023 20:04
* Add currency
* Add JS Bindings
* Add 'getAddress'
* TODO: Accounts, Send
* Accounts synchronization
* Recieve
* Send
TODO:
* Implement additional settings for token sending
* Gas fee (gas price, gas limit)
* Additional settings (sequence number, exp. timestamp)
* TODO: add texts to the dictionary
- Get transactions from the indexer by its version
- Get both transaction types Receive/Send
- Use Aptos API
* feat(aptos): replace deprecated aptos package

* feat(aptos): update aptos types

* chore: update aptos api

* feat(aptos): update aptos package

* feat(aptos): update aptos package

* fix(aptos): get network and indexer api url from network

* fix(aptos): account address issues

* refactor: set private methods

* refactor: build transaction module and api

* refactor: fix logic and constants

* fix: set aptos client with our url paths.

* test: fix imports from old package

* fix: payload type

* fix: logic code and tests
* test: adding aptos e2e send test

* feat(aptos): send test

* feat(aptos): uncomment previous tests

* Aptos fixing: History isn't full (#8512)

extending tx to ops with delegation batches and events

* fix(aptos): amount balance (#8462)

* Removing Delegated Balance from the Aptos Footer.

* Prettify addition.

* Removing unecessary code.

* Removing uneeded imports

* Fixing prettier issues.

* Removing the footer and the translation units.

* feat(aptos): add coin feature flag (#8498)

* feat(aptos): add coin feature flag

* feat(aptos): add coin feature flag for testnet

* hw-app-aptos.js moved to lib/ledgerjs/packages/hw-app-aptos. Test file has been copied, but is still not active.

* Repurposed README.md from algorand. removed some unused dependencies from the package.

* Small cosmetic changes.

* Managed to make tests work.

* Feedback from PR.

* chore: add xray ticket id

---------

Co-authored-by: Pedro Semeano <[email protected]>
Co-authored-by: Oleksii <[email protected]>
Co-authored-by: JoΓ£o Martins <[email protected]>
Co-authored-by: JoΓ£o GuimarΓ£es <[email protected]>
Co-authored-by: JoΓ£o Martins <[email protected]>
* chore: save work

* chore: save work

* test: index api
Added a test file for the LedgerAccount based on Jest.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli common Has changes in live-common desktop Has changes in LLD ledgerjs Has changes in the ledgerjs open source libs mobile Has changes in LLM tools Has changes in tools translations Translation files have been touched ui Has changes in the design system library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants