-
Notifications
You must be signed in to change notification settings - Fork 302
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
[Coin Modularization] Tezos #6754
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 5 Ignored Deployments
|
caac4d9
to
1ac320a
Compare
1ac320a
to
dbb48b0
Compare
f8559cd
to
1d8675c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok for hub scope
[Bot] Testing with 'Nitrogen' ❌ 1 txs ($4.87) ⏲ 70.6s
❌ 1 mutation errors
|
Spec (accounts) | State | Remaining Runs (est) | funds? |
---|---|---|---|
Tezos (3) | 156 ops , 5.28869 XTZ ($4.87) | 👍 182 | tz1aDK1uFAmnUXZ7KJPEmcCEFeYHiVZ56zVF |
Tezos 1 cross: 5.28869 XTZ (152ops) (tz1aDK1uFAmnUXZ7KJPEmcCEFeYHiVZ56zVF on 44'/1729'/0'/0') tezbox#0 js:2:tezos:0240051fc51799e60dcc8870415b87fc4fd948e71b23fdc0d9b8ac7438cf7d4708:tezbox
Tezos 2: 0 XTZ (4ops) (tz1he4fPXP3c9fFrztYT3k7KyYuLb28arFNn on 44'/1729'/1'/0') tezbox#1 js:2:tezos:02fe3d777af5380ef0a431c4985772c9669743050cee5feff717c3c3272d7a2810:tezbox
Tezos 3: 0 XTZ (0ops) (tz1SApkt3kmMaqNE1qtgADc6m3B49HZkFVDA on 44'/1729'/2'/0') tezbox#2 js:2:tezos:029d7bcf10737806147b22ba4578747ce4ac53e26b443c9eb1ac0e4d5bfbb8f67e:tezbox
Performance ⏲ 70.6s
Time spent for each spec: (total across mutations)
Spec (accounts) | preload | scan | re-sync | tx status | sign op | broadcast | test | destination test |
---|---|---|---|---|---|---|---|---|
TOTAL | 231ms | 3.8s | 0.67ms | 902ms | N/A | N/A | N/A | N/A |
Tezos (2) | 231ms | 3.8s | 0.67ms | 902ms | N/A | N/A | N/A | N/A |
What is the bot and how does it work? Everything is documented here!
b666da7
b666da7
to
eec3be5
Compare
2bfd9e1
to
6f181b2
Compare
ba02bd0
to
66e571a
Compare
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
7e086fc
to
4c051c6
Compare
Signed-off-by: Stéphane Prohaszka <[email protected]>
Signed-off-by: Stéphane Prohaszka <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙈
* feat: extract tezos coin Signed-off-by: Stéphane Prohaszka <[email protected]> * fix: compile issue Signed-off-by: Stéphane Prohaszka <[email protected]> * feat: update llm and lld import from tezos Signed-off-by: Stéphane Prohaszka <[email protected]> * chore: changeset Signed-off-by: Stéphane Prohaszka <[email protected]> * fix: test import error Signed-off-by: Stéphane Prohaszka <[email protected]> * fix: missing export path Signed-off-by: Stéphane Prohaszka <[email protected]> * fix: missing type definition for tezos method Signed-off-by: Stéphane Prohaszka <[email protected]> * fix: unimported Signed-off-by: Stéphane Prohaszka <[email protected]> * fix: missing pnpm-lock Signed-off-by: Stéphane Prohaszka <[email protected]> * fix: transport lost during signOperation Signed-off-by: Stéphane Prohaszka <[email protected]> * fix: solana generic issue Signed-off-by: Stéphane Prohaszka <[email protected]> * chore: add some tests Signed-off-by: Stéphane Prohaszka <[email protected]> * fix: linter Signed-off-by: Stéphane Prohaszka <[email protected]> * chore: remove cryptoassets unused Signed-off-by: Stéphane Prohaszka <[email protected]> * fix: unimported Signed-off-by: Stéphane Prohaszka <[email protected]> * chore: split baker integration test from bridge integ test Signed-off-by: Stéphane Prohaszka <[email protected]> * chore: feedbacks Signed-off-by: Stéphane Prohaszka <[email protected]> * fix: revert coin-fmk remove of areAllOperationsLoaded Signed-off-by: Stéphane Prohaszka <[email protected]> * chore: try to fix pnpm-lock Signed-off-by: Stéphane Prohaszka <[email protected]> * fix: update pnpm-lock... again Signed-off-by: Stéphane Prohaszka <[email protected]> * fix: revert changes with pnpm dedupe Signed-off-by: Stéphane Prohaszka <[email protected]> --------- Signed-off-by: Stéphane Prohaszka <[email protected]>
✅ Checklist
npx changeset
was attached.📝 Description
Tezos in its own package.
The code has also been reorganise to move forward like Polkadot.
Some issue are still to discuss:
Ledger-signer
package is used inprepareTransaction
ledgerhq/hw-transport
anoverrides
directive has been added topackage.json
fileencodeAddress
has "FIXME" to see as previous code didn't pass Typescript strict checkingAs a side effect to this PR,This feature has been separated to a different PRSignerContext
has been simplified (to be more like the dedicated Bitcoin definition). Remove of the return composition typeCoinAddress
andCoinSignature
as a mandatory declaration.Thus other coin-modules contains this modification (no impact in their behaviour)
❓ Context
🧐 Checklist for the PR Reviewers