Skip to content

fix: support multiple lines in transaction note editing - #1361

Open
sneurlax wants to merge 4 commits into
stagingfrom
fix/305-multiline-transaction-notes
Open

sneurlax wants to merge 4 commits into
stagingfrom
fix/305-multiline-transaction-notes

style: update formatting for note editor views

fa1b598
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Sep 25, 2026 in 1s

6.93% (+0.20%) compared to 03ad880

View this Pull Request on Codecov

6.93% (+0.20%) compared to 03ad880

Details

Codecov Report

❌ Patch coverage is 68.96552% with 18 lines in your changes missing coverage. Please review.
✅ Project coverage is 6.93%. Comparing base (03ad880) to head (fa1b598).
⚠️ Report is 24 commits behind head on staging.

Files with missing lines Patch % Lines
.../wallet_view/transaction_views/edit_note_view.dart 65.85% 14 Missing ⚠️
lib/pages/exchange_view/edit_trade_note_view.dart 76.47% 4 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           staging    #1361      +/-   ##
===========================================
+ Coverage     6.72%    6.93%   +0.20%     
===========================================
  Files         1118     1122       +4     
  Lines       111939   112372     +433     
===========================================
+ Hits          7533     7792     +259     
- Misses      104406   104580     +174     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.