[6.x][WIP] Bassets 🐶

[promatik]

Solves: Laravel-Backpack/community-forum#116 (using Option 7)
Status: WIP (80%)

TLDR: instead of using NPM to pull the CSS and JS assets we need, then publish them, then re-publish... we point directly to assets on a CDN; the first time they're needed, they are "internalized" (downloaded to the app's server) so that they're actually used from there; this solves bloat issues, GDPR issues, loading time issues, inconvenience issues;

In order to get it working you need to use the bassets branch on all repos. Here are their respective PRs:

• demo - [6.x] Bassets demo#449 <---- has installation instructions
• assets - Bassets DigitallyHappy/assets#15
• PRO - https://github.com/Laravel-Backpack/PRO/pull/110
• CRUD - this one

TODO: In order to consider this DONE, we need to resolve all the inline comments we added here.

[pxpm]

Hey @promatik please have a look at Laravel-Backpack/basset#35 issue.

I think it's a valid use case and our assets should support it from start.
I am not saying that we should implement the full support for it now, just leave open in the assets that someday we would probably need to set nounce="xxx" in the assets links.

👍

[tabacitu]

This is very VERY promising. Thanks a lot for doing the heavy lifting on this @promatik ! I don't want to brag too much about this (yet 😅), but... this is quite an big innovation we've done here! It's a freakin' game-changer. I expected it to work the way I imagined it but now that it does... 🤯 man, I'm excited 😅

Though... we do have quite a few things quirks to iron out.
So let's do it, let's see if we can turn this into a 100% solution!

[tabacitu]

Update: I've continued this branch, it's now as clean as it can get (without provoking huge changes in diffs). We've now removed ALL of the JS and CSS, and are only loading it using Bassets.

This does spark a few problems though. But we can consider them minor / solvable. See the inline comments above. After those are solved, we can do another round of testing... re-merge and close this 💪

[tabacitu]

Merged into the v6 branch. We're done here 🎉

#4860

[tabacitu]

Keeping this PR just for the diff.

[scrutinizer-notifier]

The inspection completed: 89 new issues