Skip to content
This repository has been archived by the owner on Nov 28, 2018. It is now read-only.

Modify installation instructions #210

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cachedout
Copy link
Contributor

I'm brand-new to mention-bot so please forgive me if I am incorrect. :]

Following the defualt instructions did not work for me. I couldn't find
any instance in which SSL support on the mention-bot end is supported.
I had to disable it in the GitHub hook for mention-bot to work.

I also noted that the default URL for the GitHub webhook needs to be
TCP/5000.

I'm brand-new to mention-bot so please forgive me if I am incorrect. :]

Following the defualt instructions did not work for me. I couldn't find
any instance in which SSL support on the mention-bot end is supported.
I had to disable it in the GitHub hook for mention-bot to work.

I also noted that the default URL for the GitHub webhook needs to be
TCP/5000.
@mention-bot
Copy link

@cachedout, thanks for your PR! By analyzing the history of the files in this pull request, we identified @vjeux, @Daniel15 and @gyandeeps to be potential reviewers.

@facebook-github-bot
Copy link

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at [email protected]. Thanks!

If you are contributing on behalf of someone else (eg your employer): the individual CLA is not sufficient - use https://developers.facebook.com/opensource/cla?type=company instead. Contact [email protected] if you have any questions.

@facebook-github-bot
Copy link

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@cachedout
Copy link
Contributor Author

Anybody going to merge this? :]

@dijonkitchen
Copy link

@cachedout the existing link is the proper link for people to utilize the same hosted bot. It's only your own URL if you run your own.

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

Successfully merging this pull request may close these issues.

4 participants