Conversation
# Conflicts: # DashSync.podspec # DashSync/shared/DashSync.m # DashSync/shared/Models/Chain/DSChain.m # DashSync/shared/Models/DAPI/DSPlatformDocumentsRequest.m # DashSync/shared/Models/DAPI/Networking/DSDAPIPlatformNetworkService.m # DashSync/shared/Models/DAPI/Networking/DSDAPIPlatformNetworkServiceProtocol.h # DashSync/shared/Models/Derivation Paths/DSFundsDerivationPath.m # DashSync/shared/Models/Identity/DSBlockchainIdentity.m # DashSync/shared/Models/Managers/Chain Managers/DSKeyManager.m # DashSync/shared/Models/Managers/Chain Managers/DSMasternodeManager.m # DashSync/shared/Models/Managers/Chain Managers/DSTransactionManager.m # DashSync/shared/Models/Masternode/DSMasternodeListDiffService.m # DashSync/shared/Models/Masternode/DSMasternodeListService.h # DashSync/shared/Models/Masternode/DSMasternodeListService.m # DashSync/shared/Models/Platform/Transitions/BlockchainIdentity/DSBlockchainIdentityCloseTransition.m # DashSync/shared/Models/Platform/Transitions/BlockchainIdentity/DSBlockchainIdentityUpdateTransition.m # DashSync/shared/Models/Transactions/Base/DSAssetLockTransaction.m # DashSync/shared/Models/Transactions/Base/DSTransaction.h # DashSync/shared/Models/Transactions/Base/DSTransaction.m # DashSync/shared/Models/Transactions/Provider/DSProviderRegistrationTransaction.m # DashSync/shared/Models/Wallet/DSAccount.h # DashSync/shared/Models/Wallet/DSAccount.m # DashSync/shared/Models/Wallet/DSWallet+Protected.h # DashSync/shared/Models/Wallet/DSWallet.h # DashSync/shared/Models/Wallet/DSWallet.m # Example/DashSync.xcodeproj/project.pbxproj # Example/DashSync/Base.lproj/Main.storyboard # Example/Podfile.lock
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
# Conflicts: # DashSync/shared/Models/Identity/DSBlockchainIdentity.h # DashSync/shared/Models/Identity/DSBlockchainIdentity.m # DashSync/shared/Models/Identity/DSBlockchainInvitation.m # DashSync/shared/Models/Identity/DSInvitation.h
Syn-McJ
left a comment
There was a problem hiding this comment.
Looks good. Some build issues in the wallet with this version of DashSync, but they don't look too bad.
This pull request includes multiple changes across various files to improve the DashSync codebase. The most important changes involve dashpay + ferment + coinjoin integration, removing a submodule, updating dependencies, and refactoring code (see also dash-shared-core changes).
Submodule Removal:
.gitmodules: Removed theplatformsubmodule.Dependency Updates:
DashSync.podspec: Updated the source URL fordashsync-iOSand downgraded theDashSharedCoredependency version. Removed dependencies onDAPI-GRPCandTinyCborObjc. [1] [2]Code Refactoring:
DashSync/iOS/Models/Managers/Service Managers/Auth/Controllers/DSBasePinViewController.m: ReplacedNSString stringWithFormatwithDSLocalizedFormatfor thedefaultTitlemethod.DashSync/shared/Categories/BigIntTypes.h: Added a new macro definition fordsutxo_hex.DashSync/shared/Categories/NSArray+Dash.handDashSync/shared/Categories/NSArray+Dash.m: Added new methods for converting between Objective-C arrays and various C++ collections. [1] [2] [3]Header and Import Adjustments:
DashSync/shared/Categories/NSData/NSData+DSHash.m: Replaceddash_shared_core.hwithdash_spv_apple_bindings.hand updated theblake3Datamethod.DashSync/shared/Categories/NSData/NSData+DSMerkAVLTree.handDashSync/shared/Categories/NSData/NSData+DSMerkAVLTree.m: Commented out imports and method implementations related toDSPlatformTreeQueryandNSData+DSCborDecoding. [1] [2]These changes collectively aim to streamline the codebase, update dependencies, and improve maintainability.