-
Notifications
You must be signed in to change notification settings - Fork 662
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
Reapply "Bower to npm" #8253
Reapply "Bower to npm" #8253
Conversation
@nschonni what kind of update we need to apply for the theme build script? |
Not sure exactly, but I think mainly it comes from how the demos were using the jQuery version from the Bower file to generate the CDN links. |
It's a bit of a big change.... |
Can, one of you, provide instructions on how we can apply that "big change" on every theme? |
I'm working on it |
As discussed at the technical review meeting, This PR are going to be merge and then new PR are going to be submitted to fix the build of GCweb, Intranet, GCWU and Base theme. |
Note that this broke the release script. The team won't be able to use the script for next versions unless the script is fixed. |
@nschonni or @LaurentGoderre when do you thing the GCWeb build would be fixed. I have a some update that I would need to apply to GCweb template next week and I need the GCWeb to be working. |
Reverts the Revert in #8228