Skip to content
This repository has been archived by the owner on Jul 17, 2019. It is now read-only.

Cloudkit backend #8

Closed

Conversation

MichaelBuckley
Copy link
Contributor

This change refactors backend services into their own classes so that users can easily adapt PunchClock to run on whatever backends they want. The backend used is configured in constants.h. Heroku is still the default backend.

It also includes a CloudKit backend. I originally wrote it as a way to learn the CloudKit framework. As I should have expected, since CloudKit is still in beta, some functionality doesn't work. I think the issues are all server-side, so once they are fixed, the CloudKit backend should just work. Aside from periodic unavailability, the issues affecting PunchClock are

  1. Assets uploaded to CloudKit through the CloudKit dashboard are not properly downloaded to the app, so custom pictures do not work. This is a known problem in the developer forums.
  2. Push notifications are only delivered when the app is running. This is also a known issue in the developer forums. Since PunchClock doesn't display push notifications in-app, this means that users will never see messages.

This change also contains a couple minor Storyboard tweaks to work around an iOS 8 UITableView display bug.

@MichaelBuckley
Copy link
Contributor Author

I have rebased this on top of the commits merged in yesterday, so it should be easier to merge.

@foozmeat
Copy link
Contributor

Hi @MichaelBuckley I merged your changes into a new cloudkit branch. I'm going to take a closer look at things and then likely will merge it into master when I'm sure it'll be working ok for us here. Thanks for your work.

@foozmeat foozmeat closed this Jul 23, 2014
@MichaelBuckley
Copy link
Contributor Author

@foozmeat That sounds like a good plan. Let me know if there's anything I can do to help. Thanks!

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

Successfully merging this pull request may close these issues.

2 participants