search

uclaradio / uclaradio.com

UCLA Radio Website
http://www.uclaradio.com
GNU Affero General Public License v3.0
31 stars 4 forks source link

Increase number of images for a blog post #416

Closed arjunr00 closed 5 years ago

arjunr00 commented 5 years ago

Digi-press requested a higher number of images per post, around 10 instead of the current limit of 5.

arjunr00 commented 5 years ago

Instead of just adding 5 more fields, I've changed the Post schema to have a single Cloudinary.Images field, which allows for an arbitrary number of images in a post. It's backwards compatible, luckily, and I've updated the code in this repository to use that field if available.

tanzeelak commented 5 years ago

@arjun-raghavan-00 does this consistently work for you? i try to upload images and press save, but the images sometimes disappear, or only one saves

arjunr00 commented 5 years ago

Oh yeah, there's a weird bug where selecting more that two images in one go only saves the first one. It works if you select one or two images, press save, select one or two more, press save, etc etc. Weird, kind of annoying bug, but I don't know if that's a problem with Keystone or our implementation of the thing?

arjunr00 commented 5 years ago

Looks like it's a Keystone issue?

arjunr00 commented 5 years ago

Closing for now; digi-press would rather keep the current limit of 5 separate images, since making blog posts is a little frustrating for them as is. Might return to this in the future, though.