Skip to content

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
* **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.

3 participants