Skip to content

Implemented getting channel by name in HTTP unicasts, i.e. GET /byname/c...#83

Merged
braice merged 1 commit intobraice:masterfrom
lobodpav:master
Jan 31, 2015
Merged

Implemented getting channel by name in HTTP unicasts, i.e. GET /byname/c...#83
braice merged 1 commit intobraice:masterfrom
lobodpav:master

Conversation

@lobodpav
Copy link
Copy Markdown
Contributor

Implemented getting channel by name in HTTP unicasts, i.e. GET /byname/channelname

braice added a commit that referenced this pull request Jan 31, 2015
Implemented getting channel by name in HTTP unicasts, i.e. GET /byname/c...
@braice braice merged commit 5e750d5 into braice:master Jan 31, 2015
@braice
Copy link
Copy Markdown
Owner

braice commented Jan 31, 2015

Thanks !

@lobodpav
Copy link
Copy Markdown
Contributor Author

Glad to help :)
There is just one issue I found with the 'space' to 'dash' replacement. If you have a channel named CT D / CT Art, then the name you have to type is a bit complex - CT-D-/-CT-Art. I have not found a better solution though :(

@braice
Copy link
Copy Markdown
Owner

braice commented Feb 28, 2015

Maybe you can escape spaces with the usual http notation %20
http://www.w3schools.com/tags/ref_urlencode.asp

@lobodpav
Copy link
Copy Markdown
Contributor Author

lobodpav commented Mar 1, 2015

Yes, but this would not eliminate long names in URLs. Was thinking of a possibility to allow users to assign aliases to channels. But I'm not sure how to achieve it in your product.
Or maybe even better, by typing URL:port where the app listens, it could display a page witl links to all channels are available.

@braice
Copy link
Copy Markdown
Owner

braice commented Mar 1, 2015

Hello

2015-03-01 6:51 GMT+01:00 Pavel Lobodinský notifications@github.com:

Yes, but this would not eliminate long names in URLs. Was thinking of a
possibility to allow users to assign aliases to channels. But I'm not sure
how to achieve it in your product.

In this case, you can specify the name in the conf and it's short enough

Or maybe even better, by typing URL:port where the app listens, it could
display a page witl links to all channels are available.

It should work in the latest version

@lobodpav
Copy link
Copy Markdown
Contributor Author

lobodpav commented Mar 1, 2015

Didn't know that. Will take a look on it. Thanks!

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.

2 participants