Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Reduce the number of queries used to build ministers index links #9418

Merged
merged 1 commit into from
Sep 6, 2024
Merged

Reduce the number of queries used to build ministers index links #9418

merged 1 commit into from
Sep 6, 2024

Commits on Sep 5, 2024

  1. Reduce the number of queries used to build ministers index links 

    The ministers index presenter was not leveraging eager loading and consequently was running huge numbers of queries. This has the potential to cause performance issues in the Sidekiq queues, as indeed was recently the case with the embassies index presenter.

We also move the govspeak helper from the main presenter to the reshuffle presenter because the main presenter doesn't use any of the methods it provides.
    @ryanb-gds
    ryanb-gds committed Sep 5, 2024
    Configuration menu
    Copy the full SHA
    615905c View commit details
    Browse the repository at this point in the history