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

Modernize sample code #14

Merged
merged 2 commits into from
Apr 21, 2017
Merged

Modernize sample code #14

merged 2 commits into from
Apr 21, 2017

Conversation

mduppes
Copy link
Contributor

@mduppes mduppes commented Apr 20, 2017

Removed parse sample code since the service no longer exists and the code is more or less the same as heroku.

Testing on heroku was rather cumbersome as you had to look at logs, so also changed the page to simply display all the received webhooks update on page load rather than "it works!"

Test Plan:

  1. Deploying code to: https://sample-webhooks-fork.herokuapp.com/
  2. Setup app secret in heroku config vars
  3. Setup a facebook webhooks subscription with callback URI: https://sample-webhooks-fork.herokuapp.com/facebook from https://developers.facebook.com/apps/{app-id}/webhooks/ with verify_url as "token"
  4. Sent a few webhooks samples and confirm they show up, e.g:

[ { "entry": [ { "time": 1492726894, "id": "1446951058", "changed_fields": [ "about" ], "uid": "1446951058" } ], "object": "user" }, { "entry": [ { "time": 1492726786, "id": "1446951058", "changed_fields": [ "about" ], "uid": "1446951058" } ], "object": "user" } ]

@adamgross42 adamgross42 merged commit 21594ec into fbsamples:master Apr 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants