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

addition to #43 - demo NodeJS webapp #102

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

Conversation

akdeniso
Copy link

@akdeniso akdeniso commented Jun 8, 2017

Simple NodeJS demo webapp instead of Java webapp for application server

@atsakiridis
Copy link
Collaborator

Thanks for the quick response @akdeniso, I just now tried to set it up locally and test but for some reason it didn't work for me. It's like the call isn't initiated when I press 'Share Screen'. Some notes:

  • Added the local extension file to my Chrome's extension
  • Tested to make sure alice and bob work in our Olympus App to make sure there are no issues with the server side
  • Started nodejs and opened 2 tabs pointing to your new sample app
  • REGISTER worked for both without issues
  • Clicking on 'Share Screen' didn't do anything. After some debugging I saw that client.js, when the function startSharing is called sourceId is undefined and I think this breaks the functionality

screen shot 2017-06-12 at 4 37 06 pm

Any ideas what might be wrong?

@akdeniso
Copy link
Author

@atsakiridis
There was an issue with an extension - it could be used only on localhost.
I have updated extension for any domain/ip.

Please, install fixed extension.

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

2 participants