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

syncedVars out, Replicants in #113

Merged
merged 45 commits into from
Apr 14, 2015
Merged

syncedVars out, Replicants in #113

merged 45 commits into from
Apr 14, 2015

Conversation

from-the-river-to-the-sea
Copy link
Contributor

No description provided.

Alex Van Camp added 30 commits April 2, 2015 17:26
first draft of synced vars rewrite, should support nested props now. …
0330acd 
…fully backwards-compatible.
completely abandoned syncedVars, we now have replicants instead.
68a7b2f 
API is also now all in one file, and is meant to be Browserified. Browserification doesn't work yet, however. None of this works, actually. Need to fix that.

[ci skip]
browserify compilation works now, need to get brfs transform working …
f2ab8d1 
…though
getting closer to making browserify work
2634bb5 
[ci skip]
further progress on new API and replicants, still can't get config li…
4ba61cb 
…b to browserfiy correctly

[ci skip]
add some missing 'assignReplicant' functionality
77f7af9 
[ci skip]
browserification now works. need to change how sockets join rooms now.
c906902 
[ci skip]
replicant instantiation now kind of works, room join maybe works idk
80b0ebb 
[ci skip]
fix emitToAll in replicator, make logging use replicant level
f6545b2 
[ci skip]
fix replicant changes not propagating correctly
706376a 
[ci skip]
[replicants] make bundle parameter optional, fix unobserve throwing a…
2de4f7d 
…n error when oldVal wasn't an object

[ci skip]
fix changes being applied to the wrong path
626ad58 
[ci skip]
[replicants] fix deubg log message
62a3345 
[ci skip]
[replicants] fix case where if the default value was an object, it wo…
de2fd15 
…uld get overwritten and not re-observed

[ci skip]
[replicants] add missing _messageHandlers property
24a8d5d 
[ci skip]
[replicants] don't digest changes if the base revision is not the latest
e033b1a 
[ci skip]
[replicants] fix replicants acting on all socket events, rather than …
a1e3e68 
…events just targeted at their specific bundle/name pair

[ci skip]
[replicants] minor cleanup
a53bac9 
[ci skip]
[replicants] if attempting to declare a replicant multiple times, ret…
0717f36 
…urn the existing replicant instead

[ci skip]
[replicants] better log message formatting for dupe prevention message
d56fd30 
[ci skip]
Merge branch 'master' into nested-synced-vars
482e65d 
Conflicts:
	package.json
[tests] get very rudimentary tests for replicants working, not all fu…
5a2566f 
…nctionality is covered yet
to make BRFS happy, generate a blank nodecg.json if one does not exist
f860469
remove node 0.10 from tests
9e0e160
whoops forgot appveyor
f7f6108
[replicants] little tweaks and flailing about in hopes of making more…
6b61247 
… tests work, which they don't! 👨
[replicants] fix assignment never working in extensions
ecc8fa8
[replicants] do server.getIO immediately rather than on server start,…
5f27de2 
… this makes things like tests actually have a chance of working
[replicants] assignment test now works
a502194
[client_includes] remove errant line
9b7c418 
[ci-skip]
Alex Van Camp and others added 15 commits April 11, 2015 23:12
[client_includes] amazingly hacky way of making sure that browserify …
0389586 
…is done doing its thing before serving `./nodecg-api.js`
[client_includes] remove leftover debug prints
89f19cc 
[ci skip]
[replicants] add tests that still need to be debugged
349bbeb 
[ci skip]
[replicants] fix extension readReplicant test (forgot it was sync)
775681c
[replicants] i'm actually mad 😠
dd7984f
[replicants] more progress on tests
e9dce23
[replicants] holy SHIT 💣
4ddfe20
[replicants] noodled around with tests and made the browser API queue…
3916882 
… assignment and change operations, executing them only after the variable has been fully declared
[replicants] add persistence (with tests)
1e14f45
[replicator] raise setMaxListeners to some absurd value
b64e512 
[ci skip]
[replicants] in-browser replicants will redeclare after reconnecting …
2e5d98f 
…to socket.io

I also wrote a test for this but I can't make it work so ¯\_(ツ)_/¯
i dont know man, fuck it
6804338 
[ci skip]
YEAH FUCK YOU JAVASCRIPT, TRIFLIN' ASS LANGUAGE 😤
6ca6aa0
@thesupremecommander
Readd old value parameter.
5f44558
Merge pull request #112 from fwdcp/nested-synced-vars-fixes
92ccc32 
Add TSC's `oldValue` fix, batch changes rather than processing one at a time
from-the-river-to-the-sea pushed a commit that referenced this pull request Apr 14, 2015
Merge pull request #113 from nodecg/nested-synced-vars
1ed85f1 
syncedVars out, Replicants in
@from-the-river-to-the-sea from-the-river-to-the-sea merged commit 1ed85f1 into v0.6.0 Apr 14, 2015
@from-the-river-to-the-sea from-the-river-to-the-sea deleted the nested-synced-vars branch April 14, 2015 18:45
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.

None yet
2 participants
@from-the-river-to-the-sea @thesupremecommander