paulw11 / Seam3

Cloudkit based persistent store for Core Data
Other
209 stars 25 forks source link

replace SMStoreNotification with Notification.Name extension #117

Closed tomaculum closed 5 years ago

tomaculum commented 5 years ago

Although this would be a breaking change for current users, I guess this makes sense since it is much more cleaner and not that complicated to adapt.

Maybe the names should be renamed to something from .syncDidFinish to .SMSyncDidFinish?

tomaculum commented 5 years ago

Yes that makes sense! I did not know that there is a way to create a deprecated warning. Did add your changes and renamed the variables accordingly.