Skip to content

Conversation

@camiloHimura
Copy link

📝 Description

Screen.Recording.2025-12-10.at.15.17.16.mov
  • Provide a clear summary of the changes and the issue that has been addressed.
  • 🛠️ Fixes # (issue number)

✨ Changes

Select what type of change your PR is:

  • 🚀 New feature (non-breaking change which adds functionality)
  • 🐞 Bug fix (non-breaking change which fixes an issue)
  • 🔄 Refactor (non-breaking change which refactors the code base)
  • ⚡ Performance improvements
  • 🎨 Style changes (code style/formatting)
  • 🧪 Tests (adding/modifying tests)
  • 📚 Documentation update
  • 📦 Build system changes
  • 🚧 CI/CD configuration
  • 🔧 Chore (general maintenance)
  • 🔒 Security update
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)

✅ Checklist

Before you submit your pull request, please make sure you have completed the following steps:

  • 📚 I have made the necessary updates to the documentation (if applicable).
  • 🧪 I have written tests that support my changes and prove that my fix is effective or my feature works (if applicable).
  • 🏷️ My PR title follows conventional commit format.

For more information about code review checklists, see the Code Review Checklist.

Signed-off-by: Colorado, Camilo <[email protected]>
Copilot AI review requested due to automatic review settings December 10, 2025 15:26
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

This PR implements pagination for the project list and refactors the load-more list component to be shared across multiple features. The changes enable infinite scrolling for projects, sources, sinks, and media items, improving performance and user experience when dealing with large datasets.

Key Changes:

  • Implemented pagination for the project list with a new useGetProjects hook that uses infinite queries
  • Increased pagination limits from 2 to 20 items for sources and sinks
  • Refactored and relocated LoadMoreList component to be a shared component across the application

Reviewed changes

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

Show a summary per file
File Description
application/ui/src/features/inspect/toolbar/sources/source-list/source-list.component.tsx Updated import path for LoadMoreList component to use shared location
application/ui/src/features/inspect/toolbar/sinks/sink-list/sink-list.component.tsx Updated import path for LoadMoreList component to use shared location
application/ui/src/features/inspect/toolbar/sinks/hooks/use-get-sources.hooks.tsx Increased sources pagination limit from 2 to 20 items
application/ui/src/features/inspect/toolbar/sinks/hooks/use-get-sinks.hooks.tsx Increased sinks pagination limit from 2 to 20 items
application/ui/src/features/inspect/toolbar/models-list/models-list.component.tsx Updated import path for LoadMoreList component to use shared location
application/ui/src/features/inspect/projects-management/projects-list.component.tsx Added pagination props and wrapped list with LoadMoreList component
application/ui/src/features/inspect/projects-management/projects-list-panel.component.tsx Integrated new hooks and pagination logic for project list
application/ui/src/features/inspect/projects-management/hooks/use-selected-project.test.tsx Added comprehensive tests for useSelectedProject hook
application/ui/src/features/inspect/projects-management/hooks/use-selected-project.hook.tsx Created hook to manage selected project with auto-fetching logic
application/ui/src/features/inspect/projects-management/hooks/use-get-project.hooks.tsx Created hook to fetch projects with infinite scroll pagination
application/ui/src/features/inspect/models/models.component.tsx Added size prop to Loading component
application/ui/src/features/inspect/dataset/media-preview/sidebar-items/sidebar-items.component.tsx Added pagination props for media items in sidebar
application/ui/src/features/inspect/dataset/media-preview/media-preview.component.tsx Added pagination props to MediaPreview component
application/ui/src/features/inspect/dataset/hooks/use-selected-media-item.test.tsx Added comprehensive tests for useSelectedMediaItem hook
application/ui/src/features/inspect/dataset/hooks/use-selected-media-item.hook.tsx Created hook to manage selected media item with auto-fetching logic
application/ui/src/features/inspect/dataset/dataset-list.component.tsx Integrated useSelectedMediaItem hook and added pagination support
application/ui/src/components/load-more-list/load-more-list.component.tsx Added padding to container styling
application/ui/mocks/mock-project.ts Created mock factory function for project testing

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

@camiloHimura camiloHimura force-pushed the ccolora11/project-list-pagination branch from c7ed0dc to b08b87a Compare December 10, 2025 15:28
Signed-off-by: Colorado, Camilo <[email protected]>
@camiloHimura camiloHimura force-pushed the ccolora11/project-list-pagination branch from b08b87a to ea85a4a Compare December 10, 2025 15:31
@camiloHimura camiloHimura merged commit 7dade8c into open-edge-platform:feature/geti-inspect Dec 11, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants