Skip to content

Change app connections list pagination order to DESC by updated column#2122

Open
rSnapkoOpenOps wants to merge 2 commits intomainfrom
feat/update-connections-table-sort
Open

Change app connections list pagination order to DESC by updated column#2122
rSnapkoOpenOps wants to merge 2 commits intomainfrom
feat/update-connections-table-sort

Conversation

@rSnapkoOpenOps
Copy link
Collaborator

Fixes OPS-3870

@linear
Copy link

linear bot commented Mar 13, 2026

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the app connections listing pagination to return the most recently updated connections first (OPS-3870), aligning paging behavior with the updated timestamp instead of the previous ascending order.

Changes:

  • Switch appConnectionService.list() pagination order to DESC.
  • Configure pagination to use the updated column as the custom pagination key.
  • Extend the unit test to assert buildPaginator is called with DESC + updated pagination.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
packages/server/api/src/app/app-connection/app-connection-service/app-connection-service.ts Changes paginator configuration to sort/page by updated in descending order.
packages/server/api/test/unit/app-connection/app-connection-service.test.ts Adds an expectation to validate the updated paginator configuration.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@sonarqubecloud
Copy link

@rSnapkoOpenOps rSnapkoOpenOps marked this pull request as ready for review March 13, 2026 14:22
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