search

Laravel-Backpack / demo

A working demo of Laravel with all Backpack packages installed.
http://backpackforlaravel.com
Other
323 stars 166 forks source link

make db:fresh commant refresh basset too #553

Closed tabacitu closed 1 year ago

tabacitu commented 1 year ago

Sometimes in our demo, basset URLs do not load what they're supposed to - see https://github.com/Laravel-Backpack/demo/issues/552 for more info.

I tested multiple things, and fixes it is running php artisan basset:fresh. So maybe we should do this after each db:fresh? 🤷‍♂️

My only problem with this is that... I don't understand why. Why does db:fresh break assets? And does it really? Because on localhost db:fresh does NOT break stuff for me. So... this fixes it... and applies the fix every hour... so it increases the changes that people don't see our demo broken... but... why?

kevin-hart-crying-gif-by-team-coco

tabacitu commented 1 year ago

Actually this might not be needed. Maybe there's a bug in basset:fresh and that's what we actually need to fix - https://github.com/Laravel-Backpack/basset/issues/70

In our demo:

Which is why the fix has always been to re-deply or re-run basset:fresh.

In the meantime, I've made our deploy script do php artisan basset:clear && php artisan basset:cache which should fix this problem. So I'll close this and reopen if I'm wrong.