-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add shop configs #179
Add shop configs #179
Conversation
Codecov Report
@@ Coverage Diff @@
## feature/magento #179 +/- ##
===================================================
- Coverage 85.2% 85.12% -0.08%
===================================================
Files 265 266 +1
Lines 7083 7155 +72
===================================================
+ Hits 6035 6091 +56
- Misses 1048 1064 +16
|
@IBOutlet private weak var tableView: UITableView! | ||
|
||
private var selectedPolicy: Policy? | ||
|
||
var tableProvider: AccountTableProvider! | ||
var isOrdersEnabled: Bool! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You inject this value to view controller and then pass it to the provider, but you can inject it directly to a provider
@@ -19,6 +19,8 @@ class HomeViewController: BaseTableViewController<HomeViewModel>, HomeTableProvi | |||
private var selectedArticle: Article? | |||
|
|||
var tableProvider: HomeTableProvider! | |||
var isPopularEnabled: Bool! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The same here, just inject config values to providers
func provider(_ provider: AccountTableProvider, didSelect policy: Policy) | ||
} | ||
|
||
class AccountTableProvider: NSObject, UITableViewDataSource, UITableViewDelegate { | ||
private let kSpaceBetweenSections: CGFloat = 10 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
private let properties inside class should not have prefix 'k'
No description provided.