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

Setup Facebook Messenger API ver 4.0 #134

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

Conversation

shnjtk
Copy link

@shnjtk shnjtk commented Oct 23, 2019

Fixed to be able to setup Facebook Messenger with API v4.0.

It shows following prompt at claudia create --configure-fb-bot command.
We need two additional parameters, page ID and subscribed fields.

Facebook page access token: <YOUR_FB_PAGE_ACCESS_TOKEN>
Facebook App Secret: <YOUR_FB_APP_SECRET>
Facebook page ID: <YOUR_FB_PAGE_ID>
Subscribed fields (comma separated): messages, messaging_postbacks

@shnjtk
Copy link
Author

shnjtk commented Oct 23, 2019

PR#133 was very helpful, thanks @prathmj

@simalexan
Copy link
Member

@shnjtk Thank you for your contribution! Much appreciated
May I ask you to add / modify the corresponding test?
If needed I can help out with some insight

@shnjtk
Copy link
Author

shnjtk commented Oct 23, 2019

@simalexan Sure, will do. I'll ask you if I need some help, thanks.

@shnjtk
Copy link
Author

shnjtk commented Oct 24, 2019

@simalexan I've added the test. Would you check?

I used process.stdin.emit and setTimeout for emulating user input.
Please let me know If you know a better way.

Here is a test coverage by nyc.

Screenshot 2019-10-25 0 52 16

@cstpalash
Copy link

Isn't merged to master?

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.

None yet

3 participants