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.

Sign up for GitHub

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

Jump to bottom

Refactor RTC connection-v2, related #17 #18

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

Refactor RTC connection-v2, related #17 #18

wants to merge 3 commits into from

Conversation

qas612820704
Copy link
Collaborator

How to run example

Installtion

$ yarn install

Get Started

  1. Run node server
DEBUG="bazaar:*" node server
  1. Open web_client/lib/bazaar-connection/test/index.html

@pastleo
Copy link
Owner

pastleo commented Jun 6, 2019

Screenshot_20190606_102750

還是要用 #4
有些東西還是 commonjs 不是給 browser 用的得包一下

@qas612820704
Copy link
Collaborator Author

I use uuid-browser (https://www.npmjs.com/package/uuid-browser).

It supposes to be used in that way.
In my scenario, it does not raise a CORS policy issue.

Or maybe we can use faker instead to generate id?

@qas612820704
Copy link
Collaborator Author

Oh, I only use uuid to generate event id.

How about afef9ea, by using Date.now() instead.

@qas612820704
Copy link
Collaborator Author

By 9af2680,

Id is generated by a count generator.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@qas612820704 @pastleo