Skip to content

Preload visualization API relations#7539

Open
KRRT7 wants to merge 4 commits into
freelawproject:mainfrom
KRRT7:preload-docket-tag-api-relations
Open

Preload visualization API relations#7539
KRRT7 wants to merge 4 commits into
freelawproject:mainfrom
KRRT7:preload-docket-tag-api-relations

Conversation

@KRRT7

@KRRT7 KRRT7 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • prefetch SCOTUSMap json_versions and clusters in VisualizationViewSet
  • avoid per-row relationship queries when VisualizationSerializer lists nested JSON versions and cluster links

Duplicate check

  • checked open PRs on July 2, 2026; adjacent preload PRs touch disclosures, people, and opinions-cited, not visualizations
  • searched open and recently merged PRs for visualization/scotusmap/jsonversion API preloading work; no matching PR found

Tests

  • uv run ruff check cl/visualizations/api_views.py
  • uv run ./manage.py test cl.visualizations.tests.APIVisualizationTestCase --keepdb could not start locally because PostgreSQL host cl-postgres does not resolve in this environment

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@mlissner

mlissner commented Jul 2, 2026

Copy link
Copy Markdown
Member

@claude review

Comment thread cl/favorites/api_views.py
@KRRT7 KRRT7 changed the title Preload docket tag API relations Preload visualization API relations Jul 2, 2026

@mlissner mlissner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Claude isn't down.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants