Trim slashes from webhook endpoint string #26

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@danielchabr

Documentation for individual bots contains examples with webhooks
containing slashes (e.g. '/webhook1234/'). This results in a path
containing two slashes. This fix makes sure that the user can use
endpoint string with or without slashes.

Trim slashes from webhook endpoint string
Documentation for individual bots contains examples with webhooks
containing slashes (e.g. '/webhook1234/'). This results in a path
containing two slashes. This fix makes sure that the user can use
endpoint string with or without slashes.
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls May 4, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling d8114a3 on Damell:master into 38f54a7 on botmasterai:master.

coveralls commented May 4, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling d8114a3 on Damell:master into 38f54a7 on botmasterai:master.

@jdwuarin

This comment has been minimized.

Show comment
Hide comment
@jdwuarin

jdwuarin May 27, 2017

Member

Support for something along those lines added in 3.1.0

Member

jdwuarin commented May 27, 2017

Support for something along those lines added in 3.1.0

@jdwuarin jdwuarin closed this May 27, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment