-
Notifications
You must be signed in to change notification settings - Fork 28
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
Convert all model objects to Decodable
and get rid of legacy codes
#62
Conversation
Converting `Country` model to extend Decodable protocol only. These changes only work on tvOS app only.
This new data fetching method is compatible with existing `Country` struct that only extends decodable.
Due to the usage of `Decodable` and `RevertSection` protocol on model objects, these 3 legacy protocol/strcuts are no longer needed: - `Collectable` protocol - `CollectableCollection` struct - `CollectableGroup` struct
Datasource class has been replaced.
Getting rid legacy data fetching and make use `MapLocationItem` that extends decodable.
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.
Refactoring is the sweetest of all development.
Nice work - when you've finished tidying up the stylistic items we discussed on slack (explicit self and fileprivate
use), feel free to merge this work.
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.
LGTM, nice work
About
This is the continuation of PR #59.
In this PR, the legacy
DataSource
class is replaced with the new one and all view controllers are updated to be compatible to use the newDataSource
implementation.Therefore, all model objects needed to be converted to extend
Decodable
.Other than that, all irrelevant and unused legacy codes are being removed from the project.
Changes
DataSource
replacement.Collectable
,CollectableGroup
,CollectableCollection
, andRequirement
.Decodable
.RevertItems
.ItemSection
addition to handle data inControlsViewController
andAlertViewController
.MapLocation
addition to represent each annotation fromMapLocations.plist
.MapAnnotationView
initiation and get rid of the legacy data fetching configuration.