Repository navigation
Add Trusted Node feature - #197
Merged
Merged
Conversation
Layout: - Refactor SettingsView's expanding rows (Blockchain, Currency, Security, Theme/Lock actions) to a consistent structure and normalize row heights via shared closedRowHeight/expandArrowSize/leadRowPad/trailRowPad constants in SettingsHelpers. - Rebuild SettingsLitecoinDetailView's peer-sync, blockchain-sync, and network-fee sections with clearer grouping, spacing, and typography (BWIPSSemiBold/BWIPSRegular, BrainwalletColor.content), and scaffold the trusted-node toggle and "Set IP Address" action. - Size the network-fee wheel picker to 40% of the row width via GeometryReader, top-align it with its "Choose preferred fee" label, and always show the blockchain detail ScrollView's scroll indicators. Network fees: - Add NewMainViewModel.currentFees, fetched via BWAPIClient.feePerKb(_:) (GET v1/fee-per-kb), and thread it through SettingsExpandingBlockchainView into SettingsLitecoinDetailView so the fee picker shows real economy/regular/luxury tiers instead of placeholder values. - Compute the fiat cost of the selected fee tier by converting litoshis to LTC and multiplying by the current exchange rate, and persist the user's choice via UserDefaults.userSetPreferredNetworkFee. - Extract the tier math (litoshis-per-tier, nearest-tier lookup for a restored preference, litoshi-to-fiat formatting) into a standalone NetworkFeeTier enum in FeeManager.swift so it's unit-testable without instantiating SwiftUI view state, and add NetworkFeeTierTests covering it. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FUKEfvG7cDWbPc1nBhwNgU
Co-Authored-By: kcw-grunt <mrkerrywashington@icloud.com>
Co-Authored-By: kcw-grunt <mrkerrywashington@icloud.com>
Co-Authored-By: kcw-grunt <mrkerrywashington@icloud.com>
Co-Authored-By: kcw-grunt <mrkerrywashington@icloud.com>
kcw-grunt
approved these changes
Sep 28, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
BRPeerManager.connect()so it pins/clears the fixed peer accordingly, and reconnects immediately when the toggle changesNote: this same set of commits was previously merged into
mainby mistake via #195 (intended target wasdevelop).mainwas left as-is per request; this PR targetsdevelopas originally intended.Test plan
xcodebuildbuild forbrainwalletscheme succeeds🤖 Generated with Claude Code