godotengine / godot-asset-library

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

all inputs reset on submit error #307

Open thmasn opened 10 months ago

thmasn commented 10 months ago

after clicking on "Submit" on the "Submit Asset" page, an error can occur and you will be sent back, with all your fields reset.

given you have to enter at least 12 different inputs to upload an asset, this can be frustrating, it happened to me twice when trying to upload my first asset.

if the error cannot be given earlier, the inputs should at least be saved as cookie or similar, to avoid re-entering all of them because one was invalid.

the specific error in my case was:

Error: Due to spam problems, we have to reject your asset because:
"https://github.com/thmasn/godot-dynamic-quality-scaling/blob/main/icon.png?raw=true" doesn't look correct; it should be similar to "http<s>://<url>.<png/jpg>".Make sure the icon URL is correct.
Please contact the community administrators if this is not spam.

a simple url paramter at the end of an image link is an unexpected error, and is a weird reason to lose 5minutes of work.

i think handling errors more gracefully would help a lot in lowering the entry barrier for uploading assets.