Skip to content
This repository has been archived by the owner on Jan 17, 2020. It is now read-only.

Session Management & text track support #19

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

Conversation

ntamblyn
Copy link

Previously if you was to join a session you would only get a console log output of this. With the new changes you are able to actually join the session and control it.

Added text track support.

@philipgiuliani
Copy link
Contributor

Hi! Thank you very much for your changes! I think you've understand something wrong in the project.

It is written in CoffeeScript, and will be translated to JavaScript when running grunt in the project folder. So the next time i'd run grunt all your changes are getting overridden. Thats also the reason why the js code looks so ugly ;) I only uploaded the JS code in the dist (distributed) folder so people dont have to build it themselves.

You think that you would be able to make the changes in the coffee files?

The commits of @PeterMHoward are ok!

In the Contribution section of the README you can find out how to get the setup done.

@ntamblyn
Copy link
Author

Yes i can do this when i have some free time :) , obviously as you can tell i just changed the code in the dist folder .

@philipgiuliani
Copy link
Contributor

I would help you out but its very hard to see the changes for me in the git diff because you reformatted the code ! :) Would be awesome. CoffeeScript is quite simple to learn. I'd love to migrate the project to es6, but i don't have time at the moment :(

@ntamblyn
Copy link
Author

Yeah sorry for the reformat was the only way i could get my head around it for the session management comparing it to the google example. All this JS is new to me and i have a long way to go. So i am obviously making rookie mistakes :). I am just going through diffs and the coffee now to point out the changes i should make.

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

Successfully merging this pull request may close these issues.

None yet

3 participants