Skip to content
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

Make Section Controllers Generic on ObjectType #625

Closed
wants to merge 3 commits into from

Conversation

Adlai-Holler
Copy link
Contributor

Changes in this pull request

  • IGListSectionController is now generic on ObjectType.
  • IGListBindingSectionController is now generic also on ViewModelType.

Issue fixed: #622

@jessesquires I can't open the iOS examples project on my machine. Xcode 8.3 complains that the project is unreadable. So I believe I updated the example projects to be compatible but I'm not 100% sure.

@facebook-github-bot
Copy link
Contributor

@Adlai-Holler updated the pull request - view changes

@jessesquires
Copy link
Contributor

@Adlai-Holler -- yep, known issue. (fixed in #631) the project file got messed up from a bad merge

@jessesquires
Copy link
Contributor

@Adlai-Holler - just landed fixes. mind rebasing?

@Adlai-Holler
Copy link
Contributor Author

Adlai-Holler commented Apr 11, 2017

Well, this thing is dead-on-arrival. As of Swift 3, you can't use a MYSectionController<MyDiffableModelClass> in place of an IGListSectionController<IGListDiffable>, even if the generic is specified as covariant, and even if you're willing to coerce with as, which is absolutely stupid.

Anyway, that's the case for now. Too bad so sad.

@Adlai-Holler Adlai-Holler deleted the MoreGenerics branch April 11, 2017 02:38
@rnystrom
Copy link
Contributor

@Adlai-Holler take a look at this, been kicking it around for a while, I think this could be really great!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Should IGListSectionController Be Generic on Object Type?
4 participants