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

board3 edit #42

Closed
wants to merge 3 commits into from
Closed

board3 edit #42

wants to merge 3 commits into from

Conversation

talonos
Copy link

@talonos talonos commented Feb 9, 2016

edited 0.3.4 to make it also work with board3.de portal extension.

@kasimi
Copy link
Contributor

kasimi commented Feb 9, 2016

Wow, this looks like it has been quite an effort! I do wonder though whether merging this is a good idea as mChat would directly depend on the board3 portal extension. If they update their code and something breaks with the mChat integration, mChat is forced to be updated as well.

Briefly glancing over the changes it looks like it could be packaged into an entirely separate "brigde extension". @talonos, do you think such a "bridge extension" would be possible, provided all the necessary events are added to mChat?

@talonos
Copy link
Author

talonos commented Feb 9, 2016

the edits don't require board3 to be installed as the code looks to see what page is loaded and portal code is only loaded if portal is present. otherwise it only loads on index.

give it a try on a test forum without board3 and its no different from the master here.

@kasimi
Copy link
Contributor

kasimi commented Feb 10, 2016

edits don't require board3 to be installed

That's a good thing but it doesn't solve the problem of a breaking change in the portal extension. It's rather another reason to create a separate extension for it, in my opinion. The majority of users don't use the portal I presume, they don't need these code edits. Forcing them to update their mChat because of a feature they don't use is not very user-friendly.

@dmzx dmzx closed this May 17, 2016
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.

None yet

3 participants