Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Setup webpack for the browser JavaScript #94

Closed
vegetabill opened this issue Oct 1, 2020 · 2 comments · Fixed by #147
Closed

Setup webpack for the browser JavaScript #94

vegetabill opened this issue Oct 1, 2020 · 2 comments · Fixed by #147
Assignees
@MariaKhantech
Labels
good first issue Good for newcomers GraceHopperOSD Reserved for GHC Open Source Day! help wanted Extra attention is needed

Comments

@vegetabill
Copy link
Collaborator

Currently, the files in public/scripts are included one by one in script tags on our HTML pages. This makes it a little tricky for interdependencies between the files and for requiring external packages.

Ideally, we'd like to build a single bundle of JavaScript that combines any necessary dependencies that are available as NPM packages.

Please look into this and configure that for our project so that we can

  • Live reload the JavaScript in development
  • Build a minified asset for the production deploy
@vegetabill vegetabill added GraceHopperOSD Reserved for GHC Open Source Day! Priority 5 Clean up tasks labels Oct 1, 2020
@vegetabill vegetabill added good first issue Good for newcomers and removed Priority 5 Clean up tasks labels Oct 1, 2020
@alodahl alodahl added the help wanted Extra attention is needed label Oct 1, 2020
@MariaKhantech
Copy link

I would like to try this

@Shraddha2104
Copy link
Collaborator

Assigned!

@alodahl alodahl mentioned this issue Oct 3, 2020
Closed
@vegetabill vegetabill mentioned this issue Oct 6, 2020
Open
vegetabill added a commit to vegetabill/keyboard-shortcuts-practice that referenced this issue Jan 9, 2021
@vegetabill
Use webpack for browser javascript. Closes Techtonica#94
47b5bd6 
  - add new libs including foreman (nf) to run webpack and express at the same time
  - move server files to ./src/server
  - move web js out of public to ./src/ui
@vegetabill vegetabill mentioned this issue Jan 9, 2021
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@MariaKhantech MariaKhantech

Labels
good first issue Good for newcomers GraceHopperOSD Reserved for GHC Open Source Day! help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
4 participants
@vegetabill @Shraddha2104 @alodahl @MariaKhantech