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

Write spec for new PROMPT_UI_DISPLAY command #186

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

fbeutin-ledger
Copy link
Contributor

No description provided.

@fbeutin-ledger fbeutin-ledger force-pushed the fbe/split_check_from_ui_display branch 4 times, most recently from 79a75d4 to 5ed169b Compare May 6, 2024 15:22
@fbeutin-ledger
Copy link
Contributor Author

PR is clean and can be review commit by commit

@fbeutin-ledger fbeutin-ledger force-pushed the fbe/split_check_from_ui_display branch 2 times, most recently from 484cb6b to defae2c Compare May 6, 2024 15:45
protocol.md Outdated Show resolved Hide resolved
@fbeutin-ledger fbeutin-ledger force-pushed the fbe/split_check_from_ui_display branch 3 times, most recently from 3020e37 to bb41ecf Compare May 17, 2024 12:55
… checks

Add companion APDUs CHECK_ASSET_IN_NO_DISPLAY and CHECK_REFUND_ADDRESS_NO_DISPLAY
Clarify CHECK_ASSET_IN as the replacement of deprecated CHECK_ASSET_IN_LEGACY for LEGACY flows too
Move FUND_LEGACY and SELL_LEGACY tests to use CHECK_ASSET_IN instead of CHECK_ASSET_IN_LEGACY
Add dedicated tests for CHECK_ASSET_IN_LEGACY
Move lib tests to use PROMPT_UI_DISPLAY
Add dedicated tests for CHECK_ASSET_X_AND_DISPLAY
@fbeutin-ledger fbeutin-ledger force-pushed the fbe/split_check_from_ui_display branch from bb41ecf to 3ac7886 Compare May 17, 2024 13:48
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.

None yet

2 participants