PR review.

.github/workflows/ci.yml

@@ -41,7 +41,7 @@ jobs:
         run : apk add --update nodejs npm
       - name : Install yarn
         run: npm install --global yarn
-      - name: Install Cardano JS Library (Ledger fork)
+      - name: Install Cardano JS Library
         run: git clone https://github.com/vacuumlabs/ledgerjs-cardano-shelley.git
       - name : Run tests
         run: |

Makefile

@@ -56,7 +56,7 @@ VARIANT_VALUES = cardano_ada
 
 DEFINES += RESET_ON_CRASH
 # Enabling DEBUG flag will enable PRINTF and disable optimizations
-DEVEL = 1
+# DEVEL = 1
 # DEFINES += HEADLESS
 
 # Enabling debug PRINTF

src/ui_nbgl.c

@@ -257,9 +257,13 @@ static void display_cancel_status(void)
 {
 	ui_idle();
 
+    // If rejectedStatus string is not NULL, then use it to display
+    // the status notification after the user has rejected the transaction.
 	if (uiContext.rejectedStatus) {
 		nbgl_useCaseStatus(uiContext.rejectedStatus, false, cancellation_status_callback);
 	}
+    // Otherwise use the statusType to determine the status to display. 
+    // The string is managed by the SDK.
 	else {
 		switch (uiContext.statusType) {
 			case STATUS_TYPE_TRANSACTION:
@@ -363,9 +367,13 @@ static void _display_choice(void)
 
 static void confirmation_status_callback(void)
 {
+    // If confirmedStatus string is not NULL, then use it to display
+    // the status notification after the user has confirmed the transaction.
 	if (uiContext.confirmedStatus) {
 		nbgl_useCaseStatus(uiContext.confirmedStatus, true, ui_idle_flow);
 		nbgl_reset_transaction_full_context();
+    // Otherwise, if the standardStatus flag is set, then use the statusType
+    // to determine the status to display (the string is managed by the SDK).
     } else if(uiContext.standardStatus) {
         switch (uiContext.statusType) {
             case STATUS_TYPE_TRANSACTION:
@@ -376,6 +384,9 @@ static void confirmation_status_callback(void)
                 break;
         }
         nbgl_reset_transaction_full_context();
+    // If neither of the above conditions are met, then the status notification
+    // display is delayed until the confirmation process is finished.
+    // A spinner is displayed in the meantime.
 	} else {
 		nbgl_useCaseSpinner("Processing");
 	}
@@ -446,6 +457,8 @@ void force_display(callback_t userAcceptCallback, callback_t userRejectCallback)
 	}
 }
 
+// Fill page content. If the content's number of lines exceeds the maximum number of lines per page,
+// the page is displayed and the pending element is added.
 void fill_and_display_if_required(const char* line1, const char* line2,
                                   callback_t userAcceptCallback,
                                   callback_t userRejectCallback)