Skip to content

refactor(sort): add sort pkg and middleware for slice architecture BED-8596#2951

Draft
mistahj67 wants to merge 1 commit into
mainfrom
BED-8596
Draft

refactor(sort): add sort pkg and middleware for slice architecture BED-8596#2951
mistahj67 wants to merge 1 commit into
mainfrom
BED-8596

Conversation

@mistahj67

Copy link
Copy Markdown
Contributor

Description

  • Ports request param sort parsing it's own agnostic package
  • Moves sort parsing into middleware to centralize bespoke boilerplate
  • Stores sort info on the bhctx for later use in db layer
  • Does not update existing sorting, should happen dynamically as endpoints are refactored to slice arch

Describe your changes in detail

Motivation and Context

Resolves BED-8596

Why is this change required? What problem does it solve?

How Has This Been Tested?

Please describe in detail how you tested your changes.
Include details of your testing environment, and the tests you ran to
see how your change affects other areas of the code, etc.

Screenshots (optional):

Types of changes

  • Chore (a change that does not modify the application functionality)

Checklist:

@mistahj67 mistahj67 self-assigned this Jul 1, 2026
@mistahj67 mistahj67 added the api A pull request containing changes affecting the API code. label Jul 1, 2026
@coderabbitai

coderabbitai Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 5f1fed7a-40b8-472f-a779-7284bc30ad3e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch BED-8596

Comment @coderabbitai help to get the list of available commands.

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

Labels

api A pull request containing changes affecting the API code.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant