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.

Sign up for GitHub

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

Jump to bottom

Stax port #1

Merged
merged 36 commits into from
Jun 14, 2023
Merged

Stax port #1

merged 36 commits into from
Jun 14, 2023

Conversation

agrojean-ledger
Copy link
Contributor

No description provided.

@agrojean-ledger
Update Makefile for Stax compatibility.
ea86328
@agrojean-ledger
Remove obsolete Blue specific code.
d32adf5
@agrojean-ledger
Add conditional compilation for Stax compatiblity.
0c3c41b
@agrojean-ledger
Move BAGL main menu UX code to separate file.
9b0ad01
@agrojean-ledger
Add Stax main menu UX.
4f7bff6
@agrojean-ledger
Move UX main menu files to specific directory.
c2c7c9e
@agrojean-ledger
Move public key BAGL UX functions to specific file.
9389717
@agrojean-ledger
Add Stax address approval UX screens.
0f2df8c
@agrojean-ledger
Rename address approval UX functions.
c20c845
@agrojean-ledger
Clean-up :
b5d24eb 
- Remove deprecated function calls,
- Use "UNUSED" macro for unused function params,
- Remove unused variables,
- Remove Blue specific structs fields init,
- Cast pointer to proper type to avoid warning.
@agrojean-ledger
Move BAGL transaction UX code to separate file.
dfa9086
@agrojean-ledger
Add Stax transaction screens.
ba0b44f
@agrojean-ledger
Rename UX transaction functions.
983ce1a
@agrojean-ledger
Add Stax glyphs.
54cebe9
@agrojean-ledger
Fix bug :
51d9e25 
Wrong structure type for message hash.
@agrojean-ledger
Clean up :
276eb5a 
- Remove obsolete PRINTF calls.
- Remove definition of obsolete Blue UX function.
- Comment macro redefinition.
- Cast variable to proper type.
@agrojean-ledger
Automate tests with Ragger.
58338b1
@agrojean-ledger
Update Stax target name and makefile.
587028d
@agrojean-ledger
Fix warnings + updates for latest SDK.
85f38a4
@agrojean-ledger agrojean-ledger changed the base branch from master to develop April 25, 2023 10:15
@agrojean-ledger agrojean-ledger force-pushed the stax-port branch 9 times, most recently from a77853b to 71b6f7f Compare April 27, 2023 10:43
sgliner-ledger
sgliner-ledger reviewed May 3, 2023
View reviewed changes
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
include/btchip_context.h Show resolved Hide resolved
src/btchip_apdu_hash_sign.c Show resolved Hide resolved
src/btchip_transaction.c Outdated Show resolved Hide resolved
src/ui/ui_main_menu_nbgl.c Outdated Show resolved Hide resolved
src/ui/ui_main_menu_nbgl.c Outdated Show resolved Hide resolved
src/ui/ui_main_menu_nbgl.c Outdated Show resolved Hide resolved
src/ui/ui_transaction_nbgl.c Outdated Show resolved Hide resolved
src/ui/ui_transaction_nbgl.c Outdated Show resolved Hide resolved
@agrojean-ledger agrojean-ledger force-pushed the stax-port branch 2 times, most recently from 5580640 to d71a01d Compare June 14, 2023 08:40
sgliner-ledger
sgliner-ledger previously approved these changes Jun 14, 2023
View reviewed changes
@agrojean-ledger agrojean-ledger merged commit faa4c09 into develop Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sgliner-ledger sgliner-ledger sgliner-ledger approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@agrojean-ledger @sgliner-ledger