ucfopen / Materia

Engage students with easily embedded apps for online courses. Supercharge your course with compelling experiences and game mechanics.
https://ucfopen.github.io/Materia-Docs/
GNU Affero General Public License v3.0
36 stars 34 forks source link

[React branch] Remove & (eventually) replace asset cache busting #1459

Closed clpetersonucf closed 1 year ago

clpetersonucf commented 1 year ago

There's an old implementation of asset cache busting via process_assets.js and alwaysload.php in the controllers. They generate a file at fuel/app/config/asset_hash.json that includes hashes for every asset served. As far as we can tell, these hashes are not put into practice: static assets served to the page do not include the hashes located in asset_hash.json. Their inclusion messes with the docker image build and release process, so they should probably be removed and eventually replaced with a different implementation of cache busting.

Note that for the purposes of the react branch, at least initially, the replacement solution is not expected.

clpetersonucf commented 1 year ago

Resolved in #1460 . A replacement for cache busting, if required, is still TBD.