search

godotengine / godot-asset-library

PHP frontend for Godot Engine's asset library
https://godotengine.org/asset-library
MIT License
285 stars 85 forks source link

Fixed missing JQuery script, changed moderator form #276

Closed Gramps closed 1 year ago

Gramps commented 1 year ago

This PR changes out the JQuery script include for a CDN version from Jquery itself. This script either does not exist on the site or my browser cannot find it for some reason. This causes the Bootstrap script to also fail when linking it. If this script loads for everyone else, then this part can be omitted.

Additionally I made some extra changes to the asset moderator form when approving / rejecting assets. Mostly formatting for readability as there was no real structure imposed on that form at all.

Calinou commented 1 year ago

We should ideally move everything away from Bower and use self-hosted files (for best performance and privacy), but this is fine for now.

Gramps commented 1 year ago

Yeah, I believe Bower is depreciated and Yarn or, I think, NPM is suggested now. Definitely self-hosted is the way to go but CDN was the best secondary choice.

coppolaemilio commented 1 year ago

Thanks!