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

Story 0 #112

Merged
merged 150 commits into from Sep 8, 2015
Merged

Story 0 #112

merged 150 commits into from Sep 8, 2015

Conversation

@yaronyg
Copy link
Member

@yaronyg yaronyg commented Sep 8, 2015

Just updating my branch with story 0's latest changes

Review on Reviewable

Tobe and others added 30 commits Jul 24, 2015
Tidy up after intergrating async bridge

Split the descriptor out from networking impl.

Ditch a load of unused event code

Refactor Peer networking into manageable chunks

Checkpoint: Solve problem of atomic update of peers

Fix function I missed

Complete the refactor of PeerNetworking

Block scope doesn't work like thats

Deal with server/client being different kinds of peer

Fix up logging, invite timeout to 60 secs

Improved logging, fixed relays

Add protected mutable dict

Fix parms mismatch

Correct protected dict implementation

Add stop method to relays

Attempt to get server to notice client has gone
yaronyg and others added 26 commits Aug 28, 2015
This primarily covers changes made to the muxer
It would only work for new cordova, not old cordova because we overwrote
the same file twice.
@yaronyg yaronyg added the in progress label Sep 8, 2015
yaronyg added a commit that referenced this pull request Sep 8, 2015
@yaronyg yaronyg merged commit 733b46f into story_0_yarong Sep 8, 2015
@yaronyg yaronyg removed the in progress label Sep 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.