Skip to content

feat: ADR-289 Explorer client scenes signed fetch - #308

Merged
LautaroPetaccio merged 8 commits into
mainfrom
feat/explorer-client-scenes-signed-fetch
Nov 18, 2025
Merged

feat: ADR-289 Explorer client scenes signed fetch#308
LautaroPetaccio merged 8 commits into
mainfrom
feat/explorer-client-scenes-signed-fetch

Conversation

@LautaroPetaccio

@LautaroPetaccio LautaroPetaccio commented Nov 7, 2025

Copy link
Copy Markdown
Contributor

This PR describes how the Explorer Client performs fetches (signed fetches to be more specific) when performing a fetch in a scene.

Closes https://github.com/decentraland/core-team/issues/123

@LautaroPetaccio
LautaroPetaccio requested a review from a team as a code owner November 7, 2025 20:15
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Nov 7, 2025

Copy link
Copy Markdown

Deploying adr with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7edb9e3
Status: ✅  Deploy successful!
Preview URL: https://062c6f34.adr-cvq.pages.dev
Branch Preview URL: https://feat-explorer-client-scenes.adr-cvq.pages.dev

View logs

Comment thread content/ADR-283-explorer-client-scenes-signed-fetch.md
Comment thread content/ADR-283-explorer-client-scenes-signed-fetch.md Outdated
@LautaroPetaccio LautaroPetaccio changed the title feat: Explorer client scenes signed fetch feat: Explorer client scenes signed fetch Nov 7, 2025
@LautaroPetaccio LautaroPetaccio changed the title feat: Explorer client scenes signed fetch feat: ADR-288 Explorer client scenes signed fetch Nov 7, 2025
@kuruk-mm

Copy link
Copy Markdown
Member

Can you rename it to ADR-289? We have the ADR-288 PR open

#307

Anyway if it is merged, we can change to ADR-289 ours

@LautaroPetaccio LautaroPetaccio changed the title feat: ADR-288 Explorer client scenes signed fetch feat: ADR-289 Explorer client scenes signed fetch Nov 10, 2025

@nachomazzara nachomazzara left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

just two minor QQ

Comment thread content/ADR-289-explorer-client-scenes-signed-fetch.md
Comment thread content/ADR-289-explorer-client-scenes-signed-fetch.md

@nachomazzara nachomazzara left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@aleortega aleortega left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGMT!

My only doubt is if we should mention something about sandboxed environment the scenes run on and also the SDK

Comment thread content/ADR-289-explorer-client-scenes-signed-fetch.md
Comment thread content/ADR-289-explorer-client-scenes-signed-fetch.md Outdated
Co-authored-by: Alejo Thomas Ortega <alejothomasortega@gmail.com>
Signed-off-by: Lautaro Petaccio <1120791+LautaroPetaccio@users.noreply.github.com>
@LautaroPetaccio
LautaroPetaccio merged commit ee717da into main Nov 18, 2025
2 checks passed
@LautaroPetaccio
LautaroPetaccio deleted the feat/explorer-client-scenes-signed-fetch branch November 18, 2025 15:21
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