-
Notifications
You must be signed in to change notification settings - Fork 50
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
4290 - Improve collection page edit #4333
base: development
Are you sure you want to change the base?
Conversation
looks fine, other than an HTR nav to nowhere! (and failing tests...) |
Still WIP. Please do not merge to dev yet. |
I've merged this into 3890. Should I delete the branch and PR or are you still working on it, @WillNigel23 ? |
'image/gif', | ||
'image/bmp', | ||
'image/tiff' | ||
].freeze |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please review if I have covered all accepted files for page images here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Documentation says: Acceptable file formats: PDF, PNG, GIF, JPG, TIF
We're not supporting PDF for an individual page upload, but we should support the others. Are we transforming them? I think we do transform some if they are uploaded in a zip file for an initial upload.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Currently our filepicker component only accepts image files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In any case, #3890 can add more supported files here
No description provided.