-
Notifications
You must be signed in to change notification settings - Fork 1
Storing puppet show state to server #14
Comments
@cvan Have you guys thought about where this is going to be hosted? It will affect how we can form the URLs. |
- Load puppet show by URL (hash, Firebase key) - Create new show if none found in URL - Button for creating a new show
...uhh, just forgot to add all these files in previous commit
@brianchirls we'll host it on a Digital Ocean droplet: nginx, Ubuntu, LetsEncrypt, Cloudflare. I can set that all up in about 30 min - I can do it sometime next week. |
@cvan Okay, if we can get it set up to do a catch-all 200 response, then we can not sure hashes in URLs and set an actual path with History API. I'm using the key I get back from Firebase for now as the unique identifier for each puppet show recording. It's not ideal 'cause they're long, case sensitive and have dashes and underscores. But it's easy and (more or less) guaranteed not to collide. Is there an easy deploy command/script for it that I can install as an npm dependency? |
yeah, do you have a good page router you use? I have a few (that actually preserve scroll position, which so few seem to do, including the popular
we could let users just load up any URL and use that slug as the key. we did something like that for https://github.com/etherworld/etherworld:
I'll have it auto-deploy based on a GitHub Webhook, so any pushed commits to |
Filed #15 for myself for early next week. |
re: webhook - very cool. There's a build process, but I think I have some scripts that will let me easily check on a branch that's just for built output. I don't need no router. It's not that complicated. I'm just parsing the URL with a regex and setting some state. And there's no scrolling. |
yeah, any commands that need to be run, just put them in a |
okay, sweet. I don't see any usage of |
- use Firebase anonymouse authentication - request authentication only when needed for editing (unless we already have it) - save user id of show creator - block editing if authenticated user doesn't match creator of show
For now, we're going to keep show state very simple, based on a single URL. CRUD, authentication, etc. will be listed under one or more separate issues and implemented as time allows.
For development purposes, there will be no authentication. Will have to discuss write permission design with @cvan and @caseyyee and make sure Firebase account is configured accordingly.
The text was updated successfully, but these errors were encountered: