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

Books import - obligatory field? #59

Open
S-Haime opened this issue Feb 28, 2024 · 1 comment
Open

Books import - obligatory field? #59

S-Haime opened this issue Feb 28, 2024 · 1 comment

Comments

@S-Haime
Copy link
Member

S-Haime commented Feb 28, 2024

When using the CSV books import function, if the 'pages' field is left blank a server error occurs. (Is the pages field somehow obligatory?)

This is not very user-friendly as this information will not always be entered nor is it logical/intuitive for it to be a required field, as it is non-essential metadata. (If this is not marked as required in the code, there is just an error happening.)

Note: Only tested on playtest, which is slightly more buggy than regular installs. Unable to confirm if it occurs elsewhere.

@S-Haime S-Haime added the bug label Feb 28, 2024
@S-Haime
Copy link
Member Author

S-Haime commented Feb 28, 2024

(Note: based on the code, it does seem fine. So it might just be a bug? )
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant