SUUpdater Notifications #234

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@rustle
Contributor
rustle commented Dec 6, 2012

SUUpdater Notifications for events that might be interesting to more than just the delegate

@andymatuschak
Contributor

Thanks for the pull request!

Hm. I am wary of this kind of API. How are you using all these?

@rustle
Contributor
rustle commented Dec 10, 2012

We attach the delegate to the app delegate or some other long lived object so it can handle messages that require a decision

-(BOOL)updaterMayCheckForUpdates:(SUUpdater *)bundle;

for example.

We then use the notifications to keep transient objects like preference UI up to date.

@andymatuschak
Contributor

Hm. Wouldn't it be better to bind the preference UI to the SUUpdater's updateInProgress key?

@rustle
Contributor
rustle commented Dec 12, 2012

Yes, for the preferences UI case bindings would probably be more correct.

@rustle rustle closed this Jan 29, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment