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

app-tezos: fix Swap flow & implement expert mode #1

Merged
merged 32 commits into from
Jan 18, 2024

Conversation

emturner
Copy link

No description provided.

ajinkyaraj-23 and others added 30 commits January 5, 2024 13:57
…reject-screen-to-blindsigning

Add reject confirm screen for blindsigning flow
…tra-id-param

Swap: do not check extra_id param anymore
…ansaction

Swap: allow one reveal before the transaction
app-exchange workflow was updated to allow overriding of app repo
- ledger-sdk version changed to 3.12.0
- It resulted in nanosp sdk version change.
- Nanosp snapshots updated due to change in font used in SDK
 - tz_operation_descriptor to tz_operation_field_descriptor
 - field to field_index
 - Use designated initializers notation: { .field=value }, this allows:
   - to have default value to false for the fields skip and complex
   - to make it easier to read
 - Use "_" in front of unused name
…lash-screen-before-parameter

Add an expert mode splash screen before the parameter field of the transaction operation
@tdejoigny-ledger tdejoigny-ledger changed the title app-tezos: fix Swap flow & implement expert mode app-tezos: fix Swap flow & implement expert mode Jan 17, 2024
…trypoint-non-complex

Parser: set transaction entrypoint parameter to non-complex.
* information
*/
static tz_parser_result
tz_step_field(tz_parser_state *state)

Check warning

Code scanning / CodeQL

Poorly documented large function Warning

Poorly documented function: fewer than 2% comments for a function of 180 lines.
@fbeutin-ledger fbeutin-ledger merged commit 527c960 into LedgerHQ:develop Jan 18, 2024
101 of 103 checks passed
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.

4 participants