-
-
Notifications
You must be signed in to change notification settings - Fork 443
Migrate to Webpacker #7464
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
Migrate to Webpacker #7464
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 48510 lines exceeds the maximum allowed for the inline comments feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 48510 lines exceeds the maximum allowed for the inline comments feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 71956 lines exceeds the maximum allowed for the inline comments feature.
2344979
to
fb23e05
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 71956 lines exceeds the maximum allowed for the inline comments feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 73396 lines exceeds the maximum allowed for the inline comments feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 102108 lines exceeds the maximum allowed for the inline comments feature.
b4d8622
to
a34c32b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 102132 lines exceeds the maximum allowed for the inline comments feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 102139 lines exceeds the maximum allowed for the inline comments feature.
20757f6
to
00205aa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 102139 lines exceeds the maximum allowed for the inline comments feature.
00205aa
to
d501a9d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 102139 lines exceeds the maximum allowed for the inline comments feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 102160 lines exceeds the maximum allowed for the inline comments feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 102162 lines exceeds the maximum allowed for the inline comments feature.
49cc732
to
eb40dcd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 102162 lines exceeds the maximum allowed for the inline comments feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 102259 lines exceeds the maximum allowed for the inline comments feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 102259 lines exceeds the maximum allowed for the inline comments feature.
6c52fe1
to
3471555
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 102259 lines exceeds the maximum allowed for the inline comments feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 102289 lines exceeds the maximum allowed for the inline comments feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 102438 lines exceeds the maximum allowed for the inline comments feature.
ab25ff1
to
8fad38c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 102469 lines exceeds the maximum allowed for the inline comments feature.
17ce54b
to
8113753
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 102502 lines exceeds the maximum allowed for the inline comments feature.
8113753
to
9ae26c5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 121869 lines exceeds the maximum allowed for the inline comments feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 121879 lines exceeds the maximum allowed for the inline comments feature.
OMG all green! Thanks @beagleknight for your help |
/** | ||
* Arabic translation for foundation-datepicker | ||
* Nico Mollet <[email protected]> | ||
*/ | ||
;(function($){ | ||
$.fn.fdatepicker.dates['ar'] = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ferblape I see all these files have been removed, where are they now? I can't see them being moved... Thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, I see foundation-datepicker
is added as a depndency in the package.json
Excellent job @ferblape, thank you so much! I left you a comment above regarding some translation files, could you check it out, please? Edit: I see it's marked as a dependency, all good! |
* develop: New Crowdin updates (#7850) Accessibility tool for development environments (#7810) Attempt to fix puffing-billy runtime error (#7853) Dynamically get all participatory space role tables for the `visible_meeting_for` query (#7855) Fix joining a meetings that the user already follows (#7854) Improve proposals import options (#7669) Fix proposal form attachment errors (#7856) New Crowdin updates (#7844) Fix branch name on generators (#7849) Conditionally render questions in election booth (#7822) Fix node version for version managers (#7848) Migrate to Webpacker (#7464)
@mrcasals @entantoencuanto @andreslucena this has been merged without any technical documentation? Or there's something that we haven't spot? |
hi @microstudi there's a second PR that hasn't been merged yet #7733 that includes documentation and a couple of guides to migrate apps and components. If you could check it and provide feedback in case you miss something I'd really appreciate it Thanks! |
Thanks @ferblape ! very helpful! |
🎩 What? Why?
This PR migrates javascripts from the old and beloved Asset Pipeline to they young and modern Webpacker
The migration has been divided in two PRs:
This migration doesn't include decidim-elections (see #7291 (comment))
This PR uses a beta version of Webpacker 6 gem, which is almost ready to be released and uses the modern Webpack v5. At the time of creating this PR that version hasn't been released yet.
The approach followed is that Decidim takes care of controlling Webpacker, so the applications using Decidim will rely on the Decidim Webpacker instance to compile the assets, this way we can manage all the dependencies in the package.json of Decidim.
Yarn has been introduced to manage frontend dependencies, and all the assets vendorized has been replaced by the npmjs version except those included in
decidim-core/app/packs/src/decidim/vendor
:Development process
The development process is very smooth, in the root folder of Decidim the webpack-dev-server is available via
bin/webpack-dev-server
and the development_app is configured to use Decidim's Webpacker, so asset changes are detected and automatic reloading is enabled.Testing environment
Node has been added to all github workflows, together with a yarn install (using a cache folder to speedup) and it's necessary to precompile the assets before running the specs.
📌 Related Issues
Testing
You should create a new development_app, start webpack dev server and start browsing:
📋 Checklist
🚨 Please review the guidelines for contributing to this repository.
docs/
.📷 Screenshots
Please add screenshots of the changes you're proposing
