Skip to content

Comments

Show sources in project overview#184

Merged
jankapunkt merged 7 commits intomainfrom
feature/projects-documents-tab
Aug 27, 2025
Merged

Show sources in project overview#184
jankapunkt merged 7 commits intomainfrom
feature/projects-documents-tab

Conversation

@jankapunkt
Copy link
Member

This lazy loads meta data of added sources for a given project, when the project is selected in the overview page.

image

@jankapunkt jankapunkt self-assigned this Aug 25, 2025
@jankapunkt jankapunkt added this to the 1.1.0 milestone Aug 25, 2025
@jankapunkt jankapunkt linked an issue Aug 25, 2025 that may be closed by this pull request
@jankapunkt jankapunkt added the live (staging) This is live and ready to be field-tested on our staging system label Aug 25, 2025

This comment was marked as outdated.

@jankapunkt jankapunkt requested a review from Copilot August 26, 2025 07:47
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 adds source metadata display to the project overview functionality. When a project is selected in the overview page, it now lazy loads and displays the sources associated with that project.

  • Adds source count display in the projects list menu with a document icon
  • Creates a detailed sources overview section in the project summary showing source name, type, and creation date
  • Updates backend to fetch and format source data for the selected project

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 3 comments.

File Description
ProjectsListMenu.vue Adds source count display with DocumentDuplicateIcon in project list entries
ProjectSummary.vue Implements sources overview section with clickable links and styling improvements
package.json Updates Vite version from 6.2.0 to 6.3.5
ProjectController.php Adds sources count to project list and detailed source metadata to project view
Files not reviewed (1)
  • web/package-lock.json: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

kadewe
kadewe previously approved these changes Aug 26, 2025
Copy link
Contributor

@kadewe kadewe left a comment

Choose a reason for hiding this comment

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

I tested on staging. Works very nice even with projects containing 60 or more source documents! Great!

@alessandrobelli alessandrobelli dismissed stale reviews from kadewe and themself via 0081747 August 27, 2025 09:51
Alessandro Belli and others added 3 commits August 27, 2025 11:53
we needed a ->withCount('sources'); inside User.php as well
@jankapunkt jankapunkt merged commit bd2480e into main Aug 27, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

live (staging) This is live and ready to be field-tested on our staging system UI/UX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Assigning documents to projects

3 participants