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

Fbe/token account owner display #104

Closed
wants to merge 16 commits into from

Conversation

fbeutin-ledger
Copy link

No description provided.

@fbeutin-ledger fbeutin-ledger force-pushed the fbe/token_account_owner_display branch from b8f3a49 to 6f4fde9 Compare December 10, 2024 13:48
@fbeutin-ledger fbeutin-ledger force-pushed the fbe/token_account_owner_display branch from 754e09a to 7866cdc Compare December 10, 2024 14:06
SIGNATURE = 0x15,
} e_tlv_tag;

typedef enum { KEY_ID_TEST = 0x00, KEY_ID_PROD = 0x03 } e_key_id;
Copy link
Collaborator

@Justkant Justkant Dec 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

KEY_ID_PROD is expected to be 0x07, I tested locally and it works with the current cal

Suggested change
typedef enum { KEY_ID_TEST = 0x00, KEY_ID_PROD = 0x03 } e_key_id;
typedef enum { KEY_ID_TEST = 0x00, KEY_ID_PROD = 0x07 } e_key_id;

@yogh333
Copy link

yogh333 commented Dec 16, 2024

Close this PR as duplicate of #99 (done after cherry-picking the 3 additional commits)

@yogh333 yogh333 closed this Dec 16, 2024
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.

3 participants