Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update create-spectacle to use import map and ESM-based one page template. #1304

Merged
merged 8 commits into from
Nov 20, 2023

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

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
7 checks passed
@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.

None yet

2 participants