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

Epi 2837 #204

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Epi 2837 #204

wants to merge 2 commits into from

Conversation

mmrj
Copy link
Contributor

@mmrj mmrj commented Apr 24, 2017

No description provided.

Molly Jones added 2 commits April 24, 2017 11:20
…ges supported

didn’t seem like this needed its own pr.
@mmrj mmrj requested a review from narenranjit April 24, 2017 18:23
@@ -171,7 +172,7 @@ ChannelManager.prototype = $.extend(ChannelManager.prototype, {
*
* **Example**
*
* var cm = new F.manager.ChannelManager();
* var cm = new F.manager.ChannelManager({shareConnection: true});
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the default example should always be the simplest - var cm = new F.manager.ChannelManager(). If it turns out that shareConnection indeed does work we'll just make that the default.

@mmrj
Copy link
Contributor Author

mmrj commented May 2, 2017

per discussion -- will make changes to this pr after https://issues.forio.com/browse/EPICENTER-2940 is complete (and we've confirmed ok to change the default for shareConnection to true).

@mmrj mmrj self-assigned this May 2, 2017
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