Skip to content

Commit bab9732

Browse files
authored
Merge pull request #5393 from selanthiraiyan/issue/4142-migrate-settings-screen-to-mvvm
Migrate `Settings` screen to MVVM
2 parents 0f43afb + 4d4fdcc commit bab9732

File tree

8 files changed

+561
-331
lines changed

8 files changed

+561
-331
lines changed

Storage/Storage/Protocols/StorageManagerType.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ public protocol StorageManagerType {
1616
///
1717
var viewStorage: StorageType { get }
1818

19-
/// Returns the `Storage` associated to the main thread.
19+
/// Returns a shared derived storage instance dedicated for write operations.
2020
///
2121
var writerDerivedStorage: StorageType { get }
2222

WooCommerce/Classes/ViewRelated/Dashboard/DashboardViewController.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -389,7 +389,7 @@ extension DashboardViewController {
389389
private extension DashboardViewController {
390390

391391
@objc func settingsTapped() {
392-
let settingsViewController = SettingsViewController(nibName: nil, bundle: nil)
392+
let settingsViewController = SettingsViewController()
393393
ServiceLocator.analytics.track(.settingsTapped)
394394
show(settingsViewController, sender: self)
395395
}

0 commit comments

Comments
 (0)