Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

upload: copy meta button #1257

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

upload: copy meta button #1257

wants to merge 4 commits into from

Conversation

jstucke
Copy link
Collaborator

@jstucke jstucke commented Sep 3, 2024

…ogging

the check_for_errors method is not useful anymore, since all non-optional form fields are 'required' (meaning you cannot click on 'submit' if they are missing) and if they are missing, we get a BadRequestKeyError anyway

the improved error logging for BadRequestKeyError will show which key is missing (otherwise it would fail silently)
replaced bootstrap selects with dropdowns and made styling more coherent
@jstucke jstucke requested a review from dorpvom September 3, 2024 14:35
@jstucke jstucke self-assigned this Sep 3, 2024
@jstucke jstucke marked this pull request as draft September 3, 2024 14:35
@jstucke jstucke force-pushed the revised-upload-page branch from a7c8f3b to 0971e71 Compare September 13, 2024 13:27
@jstucke jstucke force-pushed the revised-upload-page branch from 0971e71 to f26d91b Compare November 26, 2024 08:23
@maringuu maringuu force-pushed the revised-upload-page branch from f26d91b to efff551 Compare December 5, 2024 16:02
Base automatically changed from revised-upload-page to master December 10, 2024 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant