Julie Rymer

fix: handle broken link for the missing image file

1

We get an exception when the missing image file has previously been symlinked and for whatever reason the link got broken (it happened to me after upgrading the python version...

3 little changes (one for each commit): - in checkout template, use line object everywhere - add consistency to review file location and detail url retrieval - make every communication...

fix: typos in model field verbose name

1

There were some typos in the verbose name of some fields of Order models. I fixed them and made the appropriate change to migration files (verbose name does not actually...

fix: typo in has_children value and tests of annotated list extra info

1

There was a typo in `has_children` computation, the wrong variable is used for the comparison. I also added a test for `get_annotated_list`'s `num_to_close` and `has_children` values.

Fix basket check id and translation

1

Fixed a bug about a basket warning message that couldn't be discovered by makemessage as well as fixed the `makemessages` command in the makefile. Prevent some access to db properties...

fix: call _fetch_item from _process_image for easier override

3

In `catalogue.Importer`, we have the `_fetch_item` function as a practical override entry point. But instead of using it, `_process_image` is reimplementing the same logic directly in the function. I fixed...

fix: remove communication unused config files

1

communication app had 2 unused config file: a fully commented out one (the urls previously defined in it are already defined in the customer app) and an unused classic django...

Documentation: form accessibility reference to closed django issues

3

### Pertinent section of the Wagtail docs https://docs.wagtail.org/en/stable/advanced_topics/accessibility_considerations.html#forms ### Details In the documentation about forms accessibility, some considerations are raised that directly references django tickets. Some of those tickets have...

I propose to use `cached_property` on all, if not most of seo properties. Some properties are accessed several times in the template, and so provoke unnecessary duplicate db queries.

feat: split struct org data from SeoMixin

1

Fix #60 An alternative solution from #63 to remove struct data from `SeoMixin`, by creating `SeoOrgMixin` instead. I find this solution much less risky, the impact for the upgrade is...