Skip to content

Conversation

@daveoconnor
Copy link
Collaborator

@daveoconnor daveoconnor commented Nov 7, 2025

This is related to ticket #1996.

Makes the release reports code more readable.
Moves some processing out to celery tasks for parallelization, left items where it made more sense for developers to get querysets or objects etc. back, performance isn't the key aim.

Also adds a fix for release report generation with base path not being set correctly when generating through the admin ui.

Copy link
Member

@rbbeeston rbbeeston left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm good with this if Greg is

Copy link
Collaborator

@gregjkal gregjkal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work - one request to move a comment. Approved.

@daveoconnor daveoconnor force-pushed the doc/1996-release-reports-refactor branch 2 times, most recently from 2e64269 to d58c9c7 Compare November 12, 2025 19:54
@daveoconnor daveoconnor force-pushed the doc/1996-release-reports-refactor branch from d58c9c7 to 39007ac Compare November 18, 2025 23:30
@daveoconnor daveoconnor force-pushed the doc/1996-release-reports-refactor branch from 39007ac to a0890cb Compare November 19, 2025 01:02
@daveoconnor daveoconnor force-pushed the doc/1996-release-reports-refactor branch from 0a0d111 to 669bd2b Compare November 20, 2025 19:26
…ration error when no Algolia info exists, Updated scheme references to use constant from settings (#1895)
@daveoconnor daveoconnor force-pushed the doc/1996-release-reports-refactor branch from 669bd2b to 7d958cd Compare November 20, 2025 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants