Skip to content

fix(docs): correct SDK creation and unavailable explorer links - #183

Merged
drewstone merged 2 commits into
mainfrom
fix/sandbox-environment-examples
Sep 11, 2026
Merged

drewstone merged 2 commits into
mainfrom
fix/sandbox-environment-examples

Conversation

@drewstone

@drewstone drewstone commented Sep 11, 2026 •

Copy link
Copy Markdown
Contributor

The Sandbox quickstart's first create call used image, which the current SDK rejects. Both quickstart examples and the SDK reference now use environment, with the accepted environment values explained beside the reference example.

The endpoint gate also found that explorer.tangle.tools no longer resolves. Base Mainnet guidance now links to BaseScan. The older token table retains its addresses as text instead of linking them to the unavailable explorer or misidentifying them as Base addresses; its per-chain explorer links remain intact.

Validation: all three extracted create option objects typecheck against published @tangle-network/sandbox@0.39.0; a negative image case is rejected. Changed-file formatting, harness-copy, and live endpoint checks pass (238 URLs). The final documentation rebuild and all CI checks pass. The Netlify preview renders all three creation examples with environment and none with image. An independent agent reviewed the four-file change without findings.

@netlify

netlify Bot commented Sep 11, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for tangle-docs ready!

Name Link
🔨 Latest commit 08e7802
🔍 Latest deploy log https://app.netlify.com/projects/tangle-docs/deploys/6aa476f72185aa0008dab68f
😎 Deploy Preview https://deploy-preview-183--tangle-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@drewstone drewstone changed the title fix(sandbox): use environment in SDK creation examples fix(docs): correct SDK creation and unavailable explorer links Sep 11, 2026

@tangletools tangletools left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Independent code review completed for 08e7802 by the separate Legal adoption reviewer. No blockers in the four-file diff. The published Sandbox SDK contract supports environment for named environments, container references, and built image IDs. NetworkResources already identifies Base Mainnet chain 8453, so BaseScan matches the documented chain. The multichain token table preserves addresses without incorrectly remapping them to Base. Validation evidence supplied by the implementation owner: all three extracted create examples typecheck against Sandbox 0.39.0 with a rejected legacy image case; final documentation build and all CI checks pass; Netlify preview contains all three corrected examples. The independent review did not rerun CI or a browser. This approval records that specific completed review; it is not an automatic or blanket approval.

@drewstone
drewstone merged commit 4b7f44f into main Sep 11, 2026
11 checks passed
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.

2 participants