Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[#5152] Fix wrong resource URL after ValidationErrors#5153

Merged
smotornyuk merged 7 commits into
masterfrom
5152-wrong-resource-url
Jan 16, 2020
Merged

[#5152] Fix wrong resource URL after ValidationErrors#5153
smotornyuk merged 7 commits into
masterfrom
5152-wrong-resource-url

Conversation

@amercader
Copy link
Copy Markdown
Member

Fixes #5152

Proposed fixes:

Clears the upload and url_type fields if there is a validation error to avoid submitting the file name.
Adds a warning for users if they had previously selected a file for upload, as it's easy to miss it, creating an empty resource:

Screenshot_2020-01-10 Add resource - Test URLs 4 - CKAN

Happy to change the language being used if it doesn't seem clear enough

…Error

Also add a custom error that we can use to show a warning in the
template that users need to resubmit the file
The previously selected File is lost and it's easy to miss it, so add a
little warning under the File button warning users to resubmit it
Otherwise when using errors it is shown on the error summary on top when
using scheming
@amercader
Copy link
Copy Markdown
Member Author

Ported an adapted version to dev-v2.8 in 3607bcf. 2.7 was too complicated

fostermh pushed a commit to cioos-siooc/ckan that referenced this pull request Sep 10, 2021
@smotornyuk smotornyuk deleted the 5152-wrong-resource-url branch February 2, 2022 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Validation errors during resource creation lead to wrong URLs

2 participants