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

Special treatment for names and channels (@ and #) #9

Closed
patcon opened this issue Oct 16, 2013 · 12 comments · Fixed by #122
Closed

Special treatment for names and channels (@ and #) #9

patcon opened this issue Oct 16, 2013 · 12 comments · Fixed by #122

Comments

@patcon
Copy link
Contributor

patcon commented Oct 16, 2013

Currently, using a designation that would be hyperlinked (#channel and @username) will send a nonsensical internal id to the bot. If this is simply a matter of providing special case where, if the hyperlinked text starts with @ or #, then use the link text and not the link url:

https://github.com/tinyspeck/hubot-slack/blob/master/src/slack.coffee#L78-L79

@evansolomon
Copy link
Contributor

I don't think the issue here is the URL parsing. Names are mangled into internal ID's before the request is sent.

Any chance we could have those as plain text?

@grantmd
Copy link
Contributor

grantmd commented Oct 31, 2013

I will unmangle in the internal code that sends to hubots.

@evansolomon
Copy link
Contributor

Thanks @grantmd

@patcon
Copy link
Contributor Author

patcon commented Oct 31, 2013

Wooo! 👍

@grantmd
Copy link
Contributor

grantmd commented Nov 1, 2013

Released. Hopefully made 100% better and 0% worse.

@grantmd grantmd closed this as completed Nov 1, 2013
@evansolomon
Copy link
Contributor

@grantmd confirmed that usernames look delightfully unmangled for us

@patcon
Copy link
Contributor Author

patcon commented Nov 1, 2013

Wahoo! I shall share with the team!

@vinibaggio
Copy link

@grantmd thank you!

@patcon
Copy link
Contributor Author

patcon commented Nov 1, 2013

Ok, still having issues, should we re-open this issue to change how hubot-slack processes the @NAMEs?

https://www.evernote.com/shard/s27/sh/5a76ea20-e0e0-4468-a717-7fec572b2342/a6c4f9505bf6abff5847676958bae896

@patcon patcon reopened this Nov 1, 2013
@grantmd
Copy link
Contributor

grantmd commented Nov 1, 2013

Yeah, the @ is intentional, but I can remove it from hubot messages if you like.

@patcon
Copy link
Contributor Author

patcon commented Nov 1, 2013

Cool. If you have time, but I can also convert this to a PR later when I get some time

@ghost ghost assigned patcon Nov 1, 2013
@anandkkpr
Copy link

thanks for your efforts @grantmd!!!
thanks for sharing and making me aware of this chain @patcon!!!

++ to all above^^!!

@patcon patcon removed their assignment Dec 8, 2014
This was referenced Dec 16, 2014
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 a pull request may close this issue.

5 participants