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

Playback support. Fixes #9. #20

Closed
wants to merge 2 commits into from
Closed

Conversation

rupl
Copy link
Owner

@rupl rupl commented Jul 11, 2015

Opening a PR based on @mattgrill's fine work.

I think we need to hide some of this behind a config flag so we can easily control how/when it's used. I also need to make sure Heroku is set up right before deploying this.

  • Scenario 1: "instant mode" would bring a user to the current state of things immediately upon login. Ideally we'd poll Redis and just grab the last known state of each shape. All shapes are instantly interactive as if the person has been there all along.
  • Scenario 2: "observer mode" would play it back in real-time. The user would not be able to grab stuff until we reach the end. This mode isn't built yet but the data is theoretically being collected when this PR is merged.

Matthew Grill added 2 commits July 9, 2015 10:22
Just need Redis running locally.
@rupl rupl mentioned this pull request Jan 19, 2016
@rupl
Copy link
Owner Author

rupl commented Jan 19, 2016

what I described as "instant mode" would be the key to making a gallery as @afragon described in #18

@mattgrill
Copy link

@rupl I'll fix these conflicts.

@rupl
Copy link
Owner Author

rupl commented Jan 20, 2016

@mattgrill my hero 💕

@rupl
Copy link
Owner Author

rupl commented Sep 8, 2016

for now I am closing this. I have been brainstorming how to make this decentralized and I think I'd rather do that. No idea if I can pull it off but I will try!

@rupl rupl closed this Sep 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants