Skip to content

feat(astro): add "Download lesson as zip" button #415

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

Merged
merged 5 commits into from
Nov 15, 2024

Conversation

AriPerkkio
Copy link
Member

@AriPerkkio AriPerkkio commented Nov 13, 2024

Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@AriPerkkio AriPerkkio force-pushed the feat/download-zip branch 2 times, most recently from 623f32c to 79c37f7 Compare November 13, 2024 14:06
@AriPerkkio AriPerkkio marked this pull request as ready for review November 14, 2024 16:59
@AriPerkkio AriPerkkio requested a review from Nemikolh November 14, 2024 17:00
Copy link
Member

@Nemikolh Nemikolh left a comment

Choose a reason for hiding this comment

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

As always fantastic work on this! 🤩 Love the onClick handler at the module level 🤌

Only minor feedback!

Comment on lines +18 to +20
if (!lesson) {
throw new Error('Missing lesson');
}
Copy link
Member

Choose a reason for hiding this comment

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

Oh I suppose we are throwing here because this can only happen in development if there's a bug? 👀

Copy link
Member Author

Choose a reason for hiding this comment

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

This is only to make Typescript happy 😄

Copy link
Member

@Nemikolh Nemikolh left a comment

Choose a reason for hiding this comment

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

Nice!! 🤩

@AriPerkkio AriPerkkio merged commit 9c6e534 into stackblitz:main Nov 15, 2024
11 checks passed
@AriPerkkio AriPerkkio deleted the feat/download-zip branch November 15, 2024 08:28
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.

Easy download of the lesson code merging template + current files
2 participants