Skip to content

Various fixes and new features - #1297

Merged
rehrar merged 98 commits into
stagingfrom
vacay
Apr 29, 2026
Merged

Various fixes and new features#1297
rehrar merged 98 commits into
stagingfrom
vacay

Conversation

@sneurlax

@sneurlaxsneurlax commented Apr 8, 2026

Copy link
Copy Markdown
Member

TODO: remove (comment?) dead code
litescribe_api.dart, litescribe_response.dart, address_inscription_response.dart are now orphaned
@danrmiller

Copy link
Copy Markdown
Collaborator

I've been testing this and all good so far.

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Both branches fixed the same followers->following bug in toMap(); conflict was indentation-only.
@codecov-commenter

codecov-commenter commented Apr 28, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 1.80010% with 7801 lines in your changes missing coverage. Please review.
✅ Project coverage is 4.87%. Comparing base (e6a660c) to head (a4cd560).
⚠️ Report is 7 commits behind head on staging.

Files with missing linesPatch %Lines
lib/pages/shopinbit/shopinbit_step_4.dart0.00%1097 Missing ⚠️
...shopinbit/shopinbit_car_research_payment_view.dart0.00%468 Missing ⚠️
lib/pages/cakepay/cakepay_order_view.dart0.00%458 Missing ⚠️
lib/pages/shopinbit/shopinbit_payment_view.dart0.00%371 Missing ⚠️
lib/pages/shopinbit/shopinbit_car_fee_view.dart0.00%370 Missing ⚠️
lib/pages/shopinbit/shopinbit_shipping_view.dart0.00%368 Missing ⚠️
...b/pages/shopinbit/shopinbit_confirm_send_view.dart0.00%343 Missing ⚠️
lib/pages/cakepay/cakepay_card_detail_view.dart0.00%304 Missing ⚠️
lib/pages/shopinbit/shopinbit_ticket_detail.dart0.00%302 Missing ⚠️
lib/pages/cakepay/cakepay_confirm_send_view.dart0.00%279 Missing ⚠️
... and 44 more

❗ There is a different number of reports uploaded between BASE (e6a660c) and HEAD (a4cd560). Click for more details.

HEAD has 1 upload less than BASE
FlagBASE (e6a660c)HEAD (a4cd560)
21
Additional details and impacted files
@@ Coverage Diff @@## staging #1297 +/- ##
============================================
- Coverage 66.22% 4.87% -61.35% 
============================================
Files 14 1050 +1036 Lines 900 107031 +106131 ============================================
+ Hits 596 5217 +4621 - Misses 304 101814 +101510 

☔ View full report in Codecov by Sentry.
📢 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.

- Replace PowerShell workflow with bash-based test.yaml from tests branch
(rustup install, valac/libtss2-dev deps, git_versions stubs, bash secrets,
ensure_test_app_config.sh, prebuild.sh, build_runner, coverage checks)
- Add scripts/ensure_test_app_config.sh (stub app_config.g.dart for CI)
- Add test infrastructure: hive_ce_test_utils, mock_electrum_server,
platform_test_overrides
- Update test files from tests branch (change_now, node_service, lockscreen,
create_pin, firo_wallet, price, electrumx, cached_electrumx, node_card,
node_options_sheet, utilities)
- Delete stale mock files; regenerate all mocks via build_runner
- Add testNodeConnectionProvider + typedef to test_node_connection.dart;
switch node_card, node_options_sheet, add_edit_node_view, node_details_view
to call via provider so platform_test_overrides can intercept in tests
- Fix logger infinite recursion: swallow dispatch errors instead of
recursively calling t() which causes StackOverflow in tests
- Upgrade bitcoindart to b02aaf6c (adds isParticl param used by particl_wallet)
with dependency_overrides entry to beat bip47's pin
- Pin dart_style 3.1.3 and analyzer <8.4.0 in pubspec.lock for build_runner
- dart format node_card, node_options_sheet, add_edit_node_view,
node_details_view (format check would have failed on CI)
- Switch checkout to submodules:recursive parameter (from ci_fixes);
drops separate git submodule update step
@sneurlax

Copy link
Copy Markdown
MemberAuthor

This does currently include #1301 even tho it is a draft itself and I would recommend against including it without more work

@rehrar
rehrar merged commit 1405042 into stagingApr 29, 2026
1 of 3 checks passed
@sneurlax
sneurlax deleted the vacay branch May 27, 2026 16:02
Sign up for freeto 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.

5 participants

@sneurlax@danrmiller@codecov-commenter@rehrar