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

Enabled APNs from the server. #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

masnoble
Copy link

Files modified:

EventApp-data.sql ~Added device token table

Notifications.php ~Added “send IOS notification” button that would invoke an Ajax call to post.php (See new files)

New Files:

Store-token.php ~This handles the POST from the IOS app whenever a QR code is scanned. The APP passes in the event id and device token which is then stored in the new device token table.

Apn-2022.key/crt ~These are the certificate files that allow the server communicate with APNs

Post.php ~This is the file that will query the stored tokens and send out the notification for the devices that match the event id.

Future Work:

Removing old device entries ~There is currently no way to detect when a user no longer has an app or switched events. In either case, the device_tokens entry should be removed so that there aren’t useless rows in the table.

Invoke background check ~When we send out the notification, we may want to tell the phone to grab new event information. Currently, the app still may have to be manually refreshed to reflect new information. A good fix for this would be to make the app grab new information ever time the user opens the app.

Clean up needed notification code ~Someone should go through and remove the notification code that is no longer needed. Some of it is required by android still, but there are a few IOS specific methods/files that are no longer needed.

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.

2 participants