Skip to content

feat: dynamic lights - #1049

Merged
AlejandroAlvarezMelucciDCL merged 4 commits into
experimentalfrom
feat/dynamic-lights
Feb 19, 2025
Merged

feat: dynamic lights#1049
AlejandroAlvarezMelucciDCL merged 4 commits into
experimentalfrom
feat/dynamic-lights

Conversation

@AlejandroAlvarezMelucciDCL

@AlejandroAlvarezMelucciDCL AlejandroAlvarezMelucciDCL commented Jan 13, 2025

Copy link
Copy Markdown
Contributor

@cloudflare-workers-and-pages

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

Copy link
Copy Markdown

Deploying js-sdk-toolchain with  Cloudflare Pages  Cloudflare Pages

Latest commit: a101149
Status: ✅  Deploy successful!
Preview URL: https://6ceb6cad.js-sdk-toolchain.pages.dev
Branch Preview URL: https://feat-dynamic-lights.js-sdk-toolchain.pages.dev

View logs

@github-actions

github-actions Bot commented Jan 13, 2025

Copy link
Copy Markdown
Contributor

Test this pull request

  • The @dcl/sdk package can be tested in scenes by running

    npm install "https://sdk-team-cdn.decentraland.org/@dcl/js-sdk-toolchain/branch/feat/dynamic-lights/dcl-sdk-7.7.5-13339675953.commit-843efca.tgz"
  • To test with npx init

    export SDK_COMMANDS="https://sdk-team-cdn.decentraland.org/@dcl/js-sdk-toolchain/branch/feat/dynamic-lights/dcl-sdk-commands-7.7.5-13339675953.commit-843efca.tgz"
    npx $SDK_COMMANDS init
  • The @dcl/inspector package can be tested by visiting this url

    • Or by installing it via NPM
    npm install "https://sdk-team-cdn.decentraland.org/@dcl/js-sdk-toolchain/branch/feat/dynamic-lights/@dcl/inspector/dcl-inspector-7.7.5-13339675953.commit-843efca.tgz"
  • The /changerealm command to test test in-world

    /changerealm https://sdk-team-cdn.decentraland.org/ipfs/feat/dynamic-lights-e2e
    
  • You can preview this build entering:
    https://playground.decentraland.org/?sdk-branch=feat/dynamic-lights

@pravusjif pravusjif left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Remember to update the protocol to point to @next after your protocol branch is merged and before merging this branch.

@pravusjif

Copy link
Copy Markdown
Member

And also of course fix the CI lint (running make lint-fix in the root of the repo may help) and CI tests

@AlejandroAlvarezMelucciDCL
AlejandroAlvarezMelucciDCL changed the base branch from main to experimental January 30, 2025 01:31
@pravusjif
pravusjif requested a review from gonpombo8 February 10, 2025 13:07
Comment thread packages/@dcl/inspector/package-lock.json Outdated
Comment thread Makefile
@AlejandroAlvarezMelucciDCL
AlejandroAlvarezMelucciDCL merged commit 43029a2 into experimental Feb 19, 2025
@AlejandroAlvarezMelucciDCL
AlejandroAlvarezMelucciDCL deleted the feat/dynamic-lights branch February 19, 2025 12:57
pravusjif pushed a commit that referenced this pull request Jul 22, 2025
* Updated Makefile to improve the command update-protocol. Updated protocol to experimental branch

* Added LigthSource tests and helper method

* Regenerated playground-assets.api.md

* Updated snapshots
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.

3 participants