Skip to content

ADR 259: skybox time#293

Open
nearnshaw wants to merge 4 commits intomainfrom
skybox-time
Open

ADR 259: skybox time#293
nearnshaw wants to merge 4 commits intomainfrom
skybox-time

Conversation

@nearnshaw
Copy link
Copy Markdown
Member

No description provided.

@nearnshaw nearnshaw requested a review from a team as a code owner March 13, 2025 18:28
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Mar 13, 2025

Deploying adr with  Cloudflare Pages  Cloudflare Pages

Latest commit: a8fa355
Status: ✅  Deploy successful!
Preview URL: https://8dbd5e93.adr-cvq.pages.dev
Branch Preview URL: https://skybox-time.adr-cvq.pages.dev

View logs

Comment thread content/ADR-259-skybox-time.md
Comment thread content/ADR-259-skybox-time.md Outdated
Renamed time property to match the field supported in `scene.json`
Added examples

Signed-off-by: Alejandro Alvarez Melucci <163010988+AlejandroAlvarezMelucciDCL@users.noreply.github.com>

The current protocol supports allowing a scene to have a fixed time of day, but it doesn't allow the creators to change it on the fly.

This is a problem if we want to make time of day feel more dynamic and alive. It could be used as a narrative element, or to enhance the experience of a scene, ie: have the sun come up on queue at the end of a dj set.
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
This is a problem if we want to make time of day feel more dynamic and alive. It could be used as a narrative element, or to enhance the experience of a scene, ie: have the sun come up on queue at the end of a dj set.
This is a problem if we want to make time of day feel more dynamic and alive. It could be used as a narrative element, or to enhance the experience of a scene, ie: have the sun come up on cue at the end of a dj set.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants