LedgerHQ / app-vechain

Apache License 2.0
6 stars 11 forks source link

Tj/stax porting #14

Closed tdejoigny-ledger closed 1 year ago

tdejoigny-ledger commented 1 year ago
tdejoigny-ledger commented 1 year ago

Hello @libotony,

I will merge today if you don't have any extra comments.

libotony commented 1 year ago

Hello @libotony,

I will merge today if you don't have any extra comments.

Merge this or #13 ?

tdejoigny-ledger commented 1 year ago

14 (because #14 include also #13 commits)

libotony commented 1 year ago

I haven't finished all files, I will add some comments today, could you hold it?

tdejoigny-ledger commented 1 year ago

ok I will wait your last comments....

libotony commented 1 year ago

When do you plan to release this after this PR is merged? I have finished the review of main.c but still need some time to work on stax's UX test flow.

tdejoigny-ledger commented 1 year ago

@libotony Do you have an account on discord ? it will be easier to discuss.

libotony commented 1 year ago

@libotony Do you have an account on discord ? it will be easier to discuss.

Yes, libotony#0900