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

fix: Add generator to export image assets with their original names #29343

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

raphash
Copy link

@raphash raphash commented Jan 22, 2025

I added a snippet of generating code to export the image assets with their original names and extensions, I cleaned up the part that says you don't need to understand how to change the names automatically generated by webpack. At the end, it includes a webpack article explaining the generator code snippet.

Because

I think the changes in this commit make it easier for students to understand, it's a small and significant change.

Changes

  • Added generator to export assets with their original names.
  • Cleanup the part that says "you don't need to understand how to change the names automatically generated by webpack"
  • Added a webpack guide explaining the generator, and how u can use custom filenames when bundle.

I added a snippet of generating code to export the image assets with their original names and extensions, I cleaned up the part that says you don't need to understand how to change the names automatically generated by webpack. At the end, it includes a webpack article explaining the generator code snippet.
@github-actions github-actions bot added the Content: JavaScript Involves the JavaScript course label Jan 22, 2025
@wise-king-sullyman wise-king-sullyman requested review from a team and bycdiaz and removed request for a team January 26, 2025 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content: JavaScript Involves the JavaScript course
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants