Skip to content

Hotfix: Preload community and user for error report#2016

Merged
ArtOfCode- merged 3 commits intodevelopfrom
art/admin-error-preload
Mar 12, 2026
Merged

Hotfix: Preload community and user for error report#2016
ArtOfCode- merged 3 commits intodevelopfrom
art/admin-error-preload

Conversation

@ArtOfCode-
Copy link
Member

Admin error reports pages are currently taking ~20s to load on prod because we're hitting an N+1 query where we don't preload community. I've added this preload (and one for user because we also use that, although it doesn't seem to be causing an issue).

@ArtOfCode- ArtOfCode- requested a review from Oaphi March 12, 2026 10:29
@Oaphi
Copy link
Member

Oaphi commented Mar 12, 2026

@ArtOfCode- I've fixed up the runtime error caused by preloading communities before error types are collected.
Other than that, LGTM, the N+1 query is gone with the PR applied.

@codecov
Copy link

codecov bot commented Mar 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.76%. Comparing base (e93eb4d) to head (23ccd78).
⚠️ Report is 4 commits behind head on develop.

Additional details and impacted files
Components Coverage Δ
controllers 75.87% <100.00%> (+<0.01%) ⬆️
helpers 85.09% <ø> (ø)
jobs 66.88% <ø> (ø)
models 93.01% <ø> (ø)
tasks 61.11% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ArtOfCode- ArtOfCode- merged commit 39d0527 into develop Mar 12, 2026
13 checks passed
@ArtOfCode- ArtOfCode- deleted the art/admin-error-preload branch March 12, 2026 15:19
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.

2 participants