Skip to content

Conversation

@liam-lloyd
Copy link
Member

Currently, the order in which GET /record returns records is undefined. This remains the API contract, but as a hotfix this commit updates the endpoint to return them in the order they're listed in the request, because that is what our web app currently expects. This change should be reverted when the web app no longer relies on this behavior.

Currently, the order in which GET /record returns records is undefined.
This remains the API contract, but as a hotfix this commit updates the
endpoint to return them in the order they're listed in the request,
because that is what our web app currently expects. This change should
be reverted when the web app no longer relies on this behavior.
@codecov
Copy link

codecov bot commented Oct 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.22%. Comparing base (7c45bc7) to head (1c4cfc6).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #536   +/-   ##
=======================================
  Coverage   98.22%   98.22%           
=======================================
  Files         101      101           
  Lines        2416     2422    +6     
  Branches      400      402    +2     
=======================================
+ Hits         2373     2379    +6     
  Misses         39       39           
  Partials        4        4           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@liam-lloyd liam-lloyd merged commit d3eab9a into main Oct 21, 2025
4 checks passed
@liam-lloyd liam-lloyd deleted the order_records_as_in_the_request branch October 21, 2025 17:50
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.

3 participants