Skip to content

Conversation

@carloskelly13
Copy link
Contributor

@carloskelly13 carloskelly13 commented Nov 18, 2023

The one-page output used an outdated version of Spectacle and hard-coded in the dependencies. This implements our dynamic generation for the one-page.html option from the create-spectacle CLI and moves it over to use ESM and import maps.

How to test →

  1. Pull this PR and cd into packages/create-spectacle
  2. Either build the library or pnpm run dev
  3. Pick the options to build a new one-page deck
  4. Verify the output html file actually opens and loads

@vercel
Copy link

vercel bot commented Nov 18, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
spectacle-docs-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 20, 2023 5:51pm

@carloskelly13 carloskelly13 changed the title Update spectacle-cli to use import map and ESM-based one page template. Update create-spectacle to use import map and ESM-based one page template. Nov 18, 2023
@carloskelly13 carloskelly13 marked this pull request as ready for review November 20, 2023 15:44
Copy link
Member

@scottrippey scottrippey left a comment

Choose a reason for hiding this comment

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

Instructions worked great

@carloskelly13 carloskelly13 merged commit e7c3adb into main Nov 20, 2023
@carloskelly13 carloskelly13 deleted the feature/one-page-cli-esm branch November 20, 2023 18:01
@github-actions github-actions bot mentioned this pull request Nov 20, 2023
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