Skip to content

docs: remove broken Aptos Keyless StackBlitz demo#402

Merged
gregnazario merged 1 commit intomainfrom
cursor/aptos-keyless-demo-deactivation-77a8
Mar 20, 2026
Merged

docs: remove broken Aptos Keyless StackBlitz demo#402
gregnazario merged 1 commit intomainfrom
cursor/aptos-keyless-demo-deactivation-77a8

Conversation

@gregnazario
Copy link
Collaborator

Summary

Removes the broken StackBlitz iframe demo from the Aptos Keyless documentation pages. The embedded demo at https://stackblitz.com/edit/vitejs-vite-3fuvtu was inaccessible (access blocked), causing a poor user experience as reported in the linked issue.

Changes

  • src/content/docs/build/guides/aptos-keyless.mdx — Removed the entire "Example" section containing the broken StackBlitz iframe embed
  • src/content/docs/build/guides/aptos-keyless/simple-example.mdx — Removed the StackBlitz iframe embed and stale maintenance notice; updated page to point users to the GitHub repository with instructions to run the example locally
  • src/content/docs/build/guides/aptos-keyless/federated-keyless/simple-example.mdx — Replaced stale maintenance notice with instructions to run the example locally from the GitHub repository

Testing

  • All linting checks pass (pnpm lint)
  • Astro type checking passes (astro check --noSync, 0 errors)
  • Dev server tested: all three affected pages render correctly with no broken embeds

Closes the linked issue.

Open in Web Open in Cursor 

Removed the broken StackBlitz iframe embed from the Aptos Keyless
documentation pages. The demo was inaccessible (access blocked) and
providing a poor user experience.

Changes:
- Removed Example section with StackBlitz iframe from aptos-keyless.mdx
- Removed StackBlitz iframe and maintenance notice from simple-example.mdx
- Updated federated-keyless simple-example.mdx maintenance notice
- Updated pages to point users to the GitHub repository instead
@vercel
Copy link

vercel bot commented Mar 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
aptos-docs Ready Ready Preview, Comment Mar 20, 2026 1:05am

Request Review

@cursor cursor bot mentioned this pull request Mar 20, 2026
@gregnazario gregnazario marked this pull request as ready for review March 20, 2026 02:11
Copilot AI review requested due to automatic review settings March 20, 2026 02:11
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes a broken StackBlitz embed from the Aptos Keyless documentation and replaces it with guidance to run the examples locally from the aptos-keyless-example GitHub repository.

Changes:

  • Removed the StackBlitz iframe demo content from the main Aptos Keyless guide page.
  • Updated the Keyless Simple Example page to drop the embed/maintenance notice and direct users to local setup instructions.
  • Updated the Federated Keyless Simple Example page to replace the maintenance notice with local setup instructions.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/content/docs/build/guides/aptos-keyless.mdx Removes the broken “Example” section that embedded the StackBlitz iframe.
src/content/docs/build/guides/aptos-keyless/simple-example.mdx Removes StackBlitz embed + stale maintenance note; points to the repo README for local running instructions.
src/content/docs/build/guides/aptos-keyless/federated-keyless/simple-example.mdx Replaces stale maintenance note with instructions to run locally from the repo.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@gregnazario gregnazario merged commit 53e783a into main Mar 20, 2026
11 checks passed
@gregnazario gregnazario deleted the cursor/aptos-keyless-demo-deactivation-77a8 branch March 20, 2026 02:53
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