Skip to content

Update to OverCoat 2.0.2#12

Open
stevemoser wants to merge 1 commit intogonzalezreal:masterfrom
stevemoser:master
Open

Update to OverCoat 2.0.2#12
stevemoser wants to merge 1 commit intogonzalezreal:masterfrom
stevemoser:master

Conversation

@stevemoser
Copy link
Copy Markdown

Fixes #11

Comment thread Podfile
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please don't use the RAC sub spec unless you really want to use the ReactiveCocoa in the sample.

pod 'Overcoat', '2.0.2'

should be fine...

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The sample code references @strongify and @weakify which worked before because I think the old pod spec including one of the libraries that has that functionality. However the new pod spec for OverCoat doesn't include it and the easiest way to get things to compile is to include RAC.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess those macros were imported automatically in older versions of Mantle. With the current version you should explicitly import the right header wherever you use it:

#import <Mantle/EXTScope.h>

@gonzalezreal
Copy link
Copy Markdown
Owner

Thanks for the PR, but I think the ReadingList example does not show all the potential of Overcoat 2. This is why I created a new example for Overcoat 2.

@stevemoser
Copy link
Copy Markdown
Author

Cool, yeah I just wanted an example of saving to disk instead of an in memory Core Data store.

@gonzalezreal
Copy link
Copy Markdown
Owner

The Twitter Timeline example is saving to disk using Core Data...

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update to OverCoat 2.0.2

2 participants