search

dmecke / ChatBundle

MIT License
15 stars 13 forks source link

Allow multiple channels #8

Closed mgersten-caxy closed 10 years ago

mgersten-caxy commented 11 years ago

So, I accidentally had one piece of this in my previous PR (passing channel_id to cunningsoft_chat_post.

This PR should allow multiple channels optionally throughout the application.

mgersten-caxy commented 11 years ago

I think I still need to have a default value in the route for it to properly allow no channel, I'll look into that and let you know.

mgersten-caxy commented 11 years ago

Ok, I think this should be good now. Thanks in advance for the review!

dmecke commented 10 years ago

I am sorry it took me so long to answer! Could you please squash all these commit down to one commit? It would make the history much more readable!

mgersten-caxy commented 10 years ago

Commits squashed

mgersten-caxy commented 10 years ago

This should be ready for another review

dmecke commented 10 years ago

Thank you very much for this PR! :+1: