search

saurabhd / hk_realestate

GNU General Public License v2.0
1 stars 0 forks source link

Image restrictions #36

Closed darshi-shah closed 7 years ago

darshi-shah commented 7 years ago

We realized that there are minimal image sizes defined in the field definitions of field_object_images (in the object) and field_apart_pictures, field_apart_gallery in apartment. I already changed those because I thought they were in the database, but the values got reset somehow. There shouldn't be any restrictions. Can you have a look if you don't have anything else to do where this is and change it (probably hk_content_type feature), so we don't run into this problem again?

darshi-shah commented 7 years ago

Hi Rainer,

removed min image size restriction from field_object_images and field_apart_pictures, field_apart_gallery. You need to revert hk_ct featute.

Thanks Darshi

heliogabal commented 7 years ago

Hi Darshi, unfortunately, I realized I jumped to short to solve that problem. I had this weird bug when saving apartments, there were errors about missing images that couldn't be copied, so I had to change the filefield_path settings for location and naming of these file_fields: field_object_images, field_apart_pictures and field_apart_gallery. See the screenshots attached on the exact settings I would need incorported in the feature. field_object_images: file_setting_field_object_images

field_apart_pictures: file_settings_field_apart_pictures

field_apart_gallery: file_settings_field_apart_gallery

Please change those and update the develop branch so I can check if that is all there is to fix the problem finally. This has a high priority as it affects the clients work with the site directly, so should go in a separate hotfix push to master as soon as it has been tested.

darshi-shah commented 7 years ago

Hi Rainer,

These changes are pushed on develop branch. Please revert hk_ct feature.

Thanks

heliogabal commented 7 years ago

Hi Darshi, thanks for the fast work. Closing this, as it is incorporated in master.