What the developer wrote
Authored by julian
do not say on chain note is optional
What changed, and why it matters
This commit simply removes the word "(optional)" from two on-screen labels in the Stack Wallet app. The labels now say "On chain Note" instead of "On chain Note (optional)". There is no code behavior change, no security fix, and no vulnerability being addressed. It is a minor user-interface wording update.
No security action required. Treat as a routine UI wording change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff changes two Dart UI strings in send_view.dart and confirm_transaction_view.dart, removing the parenthetical “(optional)” from labels shown for Epiccash and Mimblewimblecoin on-chain notes. No logic, validation, data handling, or cryptographic code is modified. The change is purely cosmetic/labeling.
Changed components
lib/pages/send_view/send_view.dartlib/pages/send_view/confirm_transaction_view.dartInspect captured patch +2 / −2
diff --git a/lib/pages/send_view/confirm_transaction_view.dart b/lib/pages/send_view/confirm_transaction_view.dart
index 9a7640a..dfd6c98 100644
--- a/lib/pages/send_view/confirm_transaction_view.dart
+++ b/lib/pages/send_view/confirm_transaction_view.dart
@@ -1181,7 +1181,7 @@ class _ConfirmTransactionViewState
children: [
if (coin is Epiccash || coin is Mimblewimblecoin)
Text(
- "On chain Note (optional)",
+ "On chain Note",
style: STextStyles.smallMed12(context),
textAlign: TextAlign.left,
),
diff --git a/lib/pages/send_view/send_view.dart b/lib/pages/send_view/send_view.dart
index 8761a72..94b5663 100644
--- a/lib/pages/send_view/send_view.dart
+++ b/lib/pages/send_view/send_view.dart
@@ -2353,7 +2353,7 @@ class _SendViewState extends ConsumerState<SendView> {
const SizedBox(height: 12),
if (coin is Epiccash)
Text(
- "On chain Note (optional)",
+ "On chain Note",
style: STextStyles.smallMed12(context),
textAlign: TextAlign.left,
),
Why this scored 16/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.