Skip to content

{web,graphql}: Add pagination to pull row conflicts #509

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 23, 2025
Merged

Conversation

tbantle22
Copy link
Collaborator

No description provided.

@tbantle22 tbantle22 requested a review from Copilot June 23, 2025 13:07
Copy link
Contributor

@Copilot 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

This PR adds pagination to pull row conflicts by introducing an optional offset parameter into GraphQL types, queries, resolvers, and UI components.

  • Updates GraphQL types and schema to support an optional offset and nextOffset for pagination.
  • Introduces a new hook (useRowConflicts) and updates UI components (e.g., ConflictsTable) to implement infinite scrolling.
  • Modifies backend query factory methods, resolvers, and models to incorporate the offset parameter in SQL queries.

Reviewed Changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 1 comment.

File Description
web/renderer/gen/graphql-types.tsx Added optional offset properties and updated query types for pagination.
web/renderer/components/pageComponents/DatabasePage/ForPullConflicts/* Introduced a new hook (useRowConflicts) and updated state/UI components to support infinite scrolling.
graphql-server/* Updated query factories, resolvers, and models to accept and process the offset parameter in pagination.

@tbantle22 tbantle22 merged commit b9dd735 into main Jun 23, 2025
3 checks passed
@tbantle22 tbantle22 deleted the taylor/con-pag branch June 23, 2025 13:25
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.

1 participant