LB (Ben Johnston)
LB (Ben Johnston)
Update - this likely still exists and file content has moved to `https://github.com/wagtail/wagtail/blob/main/wagtail/images/signal_handlers.py` https://github.com/wagtail/wagtail/blob/main/wagtail/images/signal_handlers.py ```python def pre_save_image_feature_detection(instance, **kwargs): # .. add check here for kwargs['raw'] ``` See https://docs.djangoproject.com/en/4.0/ref/signals/#pre-save for `raw`...
Go for it @anujaraj - before you start on code be sure to follow the steps above and check you can reproduce the issue. Write a comment with any extra...
Go for it @4the4ryushin - be sure you can reproduce and understand the problem. I'd recommend you find where to write a unit test for this and start with writing...
Similar - except this is an images application so it needs to go in https://github.com/wagtail/wagtail/blob/main/wagtail/images/tests/test_signal_handlers.py
@4the4ryushin I encourage you to look through the code yourself and try to find either a. Fixtures that have images or b. Similar test code where we are testing code...
Partial work done via https://github.com/wagtail/wagtail/pull/9585
@DokuaAsiedu thanks for taking the time to contribute this. Can you add some description to this PR with the reasoning behind this change and why the alt text and existing...
Thanks @DokuaAsiedu - yeah I think its best to close this PR then.
Thanks @th3hamm0r code looks good but haven't had a chance to run locally.
4.1 stable f27687316cefa319b5ece5a33374db9ca0f1b858