Skip to content

Add self-hosted tutorial player (wallet-connection demo)#175

Open
kelley-sean wants to merge 1 commit into
FraxFinance:masterfrom
kelley-sean:hosted-tutorial
Open

Add self-hosted tutorial player (wallet-connection demo)#175
kelley-sean wants to merge 1 commit into
FraxFinance:masterfrom
kelley-sean:hosted-tutorial

Conversation

@kelley-sean

Copy link
Copy Markdown
Contributor

Summary

Replaces the externally hosted (Supademo) embed for the Connecting wallet tutorial with a self-hosted static player — a test of moving tutorials in-house to cut costs.

  • New self-hosted bundle under public/tutorials/, served same-origin as a static iframe (no external dependency, no third-party tracking/cookies).
  • Shared player at public/tutorials/player/ so future tutorials reuse a single copy; per-tutorial content (screenshots + demo data) lives under its own folder.
  • Only the Connecting wallet entry is switched — all other tutorials remain on Supademo so the two can be compared side by side on the same page before migrating the rest.

Testing

  • next build (static export) passes; the bundle is emitted to out/tutorials/… and the tutorials page references the local iframe.
  • All assets serve with correct MIME types.
  • Player reviewed: no innerHTML/eval, no external network calls, http(s)-only link guard, strict CSP on the iframe document.

How to review

Open /protocol/tutorialsBasics → Connecting wallet (self-hosted) and compare against the other entries (still Supademo).

Replace the externally hosted (Supademo) iframe for the "Connecting
wallet" tutorial with a self-hosted static player served from
public/tutorials/. The bundle is fully self-contained (vanilla JS
player, screenshots, and demo data) and loads as a same-origin iframe,
removing the external dependency for this tutorial.

Other tutorials remain on Supademo for now so the two can be compared
on the same page before migrating the rest.
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.

1 participant