Skip to content

Commit

Permalink
fix: summaries select_related
Browse files Browse the repository at this point in the history
  • Loading branch information
davidlougheed committed Sep 9, 2024
1 parent 80a3e45 commit 250edb5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion chord_metadata_service/experiments/summaries.py
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ async def dt_experiment_summary(
experiments = experiments.filter(dataset_id=dataset_id)
elif project_id := scope.project_id:
# Project is set but dataset isn't
experiments = experiments.select_related("project").filter(dataset__project_id=project_id)
experiments = experiments.select_related("dataset").filter(dataset__project_id=project_id)

# Parallel-gather all statistics we may need for this response
(
Expand Down
2 changes: 1 addition & 1 deletion chord_metadata_service/phenopackets/summaries.py
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ async def dt_phenopacket_summary(
phenopackets = phenopackets.filter(dataset_id=dataset_id)
elif project_id := scope.project_id:
# Project is set but dataset isn't
phenopackets = phenopackets.select_related("project").filter(dataset__project_id=project_id)
phenopackets = phenopackets.select_related("dataset").filter(dataset__project_id=project_id)

Check warning on line 96 in chord_metadata_service/phenopackets/summaries.py

View check run for this annotation

Codecov / codecov/patch

chord_metadata_service/phenopackets/summaries.py#L96

Added line #L96 was not covered by tests

# Parallel-gather all statistics we may need for this response
(
Expand Down

0 comments on commit 250edb5

Please sign in to comment.