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

available-rooms: hardcoded MUC service JID #159

Open
ge0rg opened this issue May 17, 2016 · 9 comments
Open

available-rooms: hardcoded MUC service JID #159

ge0rg opened this issue May 17, 2016 · 9 comments

Comments

@ge0rg
Copy link

ge0rg commented May 17, 2016

The MUC service that is searched by available-rooms is hard-coded to 'conference.' + Candy.Core.getConnection().domain, this should be a parameter to the init() function.

In my case, I'm running an anonymous login on 'anon.yax.im' and the MUC service domain is 'chat.yax.im', so it would be great to have support for that.

Also, the documentation does not state if the init() function should be called before or after the Candy.init. I tried both (after hardcoding the MUC service JID), and neither worked :-(

@ge0rg
Copy link
Author

ge0rg commented May 17, 2016

P.S: the createroom plugin has a subdomain option that is also based on the connection domain, making it impossible to use chat.yax.im from anon.yax.im.

@smokku
Copy link

smokku commented May 17, 2016

@benlangfeld
Copy link
Member

You might submit that as a PR, @smokku

@smokku
Copy link

smokku commented May 17, 2016

Sorry... No time for a proper PR.

@benlangfeld
Copy link
Member

You've already done the work to produce the diff, but you don't want to take literally 1 minute to contribute? That's a rather selfish attitude to take to open source, I must say.

@smokku
Copy link

smokku commented May 17, 2016

Read your Contributing guidelines and think about that 1 minute again.

I already published the diff and took time to point your users to it. If that is selfish...

@benlangfeld
Copy link
Member

benlangfeld commented May 17, 2016

Our contributing guidelines are perfectly reasonable, @smokku. They're incredibly liberal; we even offer a commit bit: https://github.com/candy-chat/candy-plugins#contributing

It's unfortunate that you believe it's reasonable to make use of the work done by contributors to this project, but object on unknown grounds to making a small contribution yourself. That's not how open source is supposed to work.

@smokku
Copy link

smokku commented May 17, 2016

I am sorry I took time to contribute to your project.
I will stop now and won't bother you again.

@benlangfeld
Copy link
Member

The negative side of a liberal license like MIT is that @smokku is perfectly within his rights to behave like this. I'm not sure where the dismissive attitude comes from. I do hope, though, that he'll take a look at the value he gets from projects like this and at some point reflect on his attitude towards those projects, and maybe take 30 seconds from his very busy schedule to try and properly distribute his changes upstream to pay it forward.

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

No branches or pull requests

3 participants