Fix 'Syncing doesn’t always show' - #10
Merged
Merged
Conversation
llbartekll
added a commit
that referenced
this pull request
Apr 13, 2026
Merges three commits from the migration-235 worktree: - feat: mirror PIN-change re-encryption into SwiftDashSDK (#10) - feat: migrate backup seed phrase display to SwiftDashSDK (#13) - feat: flip mnemonic generation to SwiftDashSDK (#3) Resolved UUID collisions in pbxproj (remapped 109-114 → 115-121 to avoid overlap with Container/WalletState UUIDs from HEAD). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
llbartekll
added a commit
that referenced
this pull request
May 14, 2026
SwiftDashSDK's WalletStorage no longer keeps a PIN-encrypted seed (mnemonics are stored as plain keychain bytes via storeMnemonic; legacy wallet.seed / wallet.pin accounts are scrubbed on launch by cleanupLegacyItems), so the PIN-change mirror is no longer required — the adapter's body had already been reduced to a logger-only no-op. Delete SwiftDashSDKPinChanger.swift and its six Xcode project references, collapse DWSetPinModel.setPin: to a thin wrapper around [DSAuthenticationManager setupNewPin:], and update DASHSYNC_MIGRATION.md rows #10 and #13 to reflect that PIN handling remains entirely a DashSync concern. No DashSync code paths modified; both dashwallet and dashpay schemes build clean on iPhone 17 simulator. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
llbartekll
added a commit
that referenced
this pull request
Jun 18, 2026
CoinJoin row #20 corrected: mixing is being dropped (legacy mixing UI already retired), not migrated. The one capability that moves over — recovering/sweeping already-mixed coins into spendable balance — is implemented on both sides via ManagedCoreWallet.sweepCoinJoinAccount (platform PR #3817, awaiting merge). CoinJoin is no longer the 'keep DashSync linked indefinitely' hard blocker. Updated the functional table, 'Where we are', Hard blockers, and Wave 6. Easy-win status promotions: - #13 Backup seed phrase: Flipped -> Done (backup read is 100% SwiftDashSDK, no fallback; adapter already retired). - #19 DPNS username lookup: Flipped -> Done (availability check fully on SDK; prefix search carved out to #18; dead request residue removed). - #10 PIN-change half: close-out note (mirror retired permanently, stays on DashSync by decision). - #7: isMinimal noted as removed. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
llbartekll
added a commit
that referenced
this pull request
Jun 24, 2026
…rator/HPMN Migrates the Masternode Keys tool (Settings -> Tools -> Show Masternode Keys) off DashSync (migration item #15): - Owner/Voting (ECDSA) now derive from SwiftDashSDK via a co-located MasternodeProviderKeyDeriver (Wallet.getAccount + Account.derivePrivateKeyWIF + managed provider address pool; hex via WIFParser). Paths match DSAuthenticationKeysDerivationPath exactly: m/9'/<5'|1'>/3'/1' (voting), m/9'/<5'|1'>/3'/2' (owner), ECDSA, soft key index. - Removed Operator (BLS) and HPMN/Platform (EdDSA): the FFI does not export their per-index public keys, which masternode registration requires. - Dropped the used / "Used at: IP" status and all DashSync from the models: no masternode-list-by-key FFI exists, so it can't be reproduced DashSync-free. PIN gate stays on the app-wide DSAuthenticationManager (migration #10). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.
No description provided.