Skip to content

Fix ordering of cached collections upon revive#1815

Merged
ArtOfCode- merged 2 commits intodevelopfrom
art/category-ordering
Sep 2, 2025
Merged

Fix ordering of cached collections upon revive#1815
ArtOfCode- merged 2 commits intodevelopfrom
art/category-ordering

Conversation

@ArtOfCode-
Copy link
Member

Moves order clauses out of fetch_collection blocks.

@Oaphi Oaphi changed the title Fix ordering Fix ordering of cached collections upon revive Sep 2, 2025
@codecov
Copy link

codecov bot commented Sep 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.27%. Comparing base (16d0278) to head (fde305c).
⚠️ Report is 3 commits behind head on develop.

Additional details and impacted files
Components Coverage Δ
controllers 70.91% <100.00%> (+0.03%) ⬆️
helpers 81.96% <ø> (ø)
jobs 60.00% <ø> (ø)
models 88.36% <ø> (ø)
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.

@Oaphi Oaphi added the area: backend Changes to server-side code label Sep 2, 2025
Copy link
Member

@Oaphi Oaphi left a comment

Choose a reason for hiding this comment

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

not sure if it's ready for review, but in any case - LGTM, tested, works as expected, and we now have a regression test on top of it, nice!

@ArtOfCode- ArtOfCode- merged commit c86515b into develop Sep 2, 2025
13 checks passed
@ArtOfCode- ArtOfCode- deleted the art/category-ordering branch September 2, 2025 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: backend Changes to server-side code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants