Skip to content

Conversation

oslego
Copy link
Contributor

@oslego oslego commented Oct 6, 2015

This PR adjusts the Makefile to export the SonicServer, SonicSocket and SonicCoder under a global Sonicnet object attached to window. This is in addition to the dependency manager hooks which browserify exposes. See details about --standalone flag.

This fixes lib/api-test.html and makes it practical to consume the library in a browser in the absence of a dependency manager.

I wasn't able to find a solution to configure browserify to export the individual objects directly to window without explicitly changing lib/main.js as attempted in @mildmojo's PR.

This changes only the build in lib/. The Makefiles for the demos are left unchanged.

@FalconerTC
Copy link

+1

@Astriel Astriel mentioned this pull request Mar 22, 2017
@Astriel
Copy link

Astriel commented Mar 22, 2017

@FalconerTC : Did you manage to make the plugin work ? I'm still having some trouble.

@ramasubbaiya
Copy link

ramasubbaiya commented Mar 28, 2017

@Astriel It did work for me. What is the issue you are having?

Copy link

@MatanYemini MatanYemini left a comment

Choose a reason for hiding this comment

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

Seems good to me

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.

5 participants