Skip to content

Development: Calculate CQI Data#51

Merged
az108 merged 41 commits intomainfrom
feature/cqi-data
Dec 10, 2025
Merged

Development: Calculate CQI Data#51
az108 merged 41 commits intomainfrom
feature/cqi-data

Conversation

@Cathy0123456789
Copy link
Copy Markdown
Collaborator

@Cathy0123456789 Cathy0123456789 commented Dec 4, 2025

Checklist

General

  • I tested all changes and their related features with all corresponding user types.

Server

  • Important: I implemented the changes with a very good performance and prevented too many (unnecessary) and too complex database calls.
  • I strictly followed the principle of data economy for all database calls.
  • I strictly followed the server coding and design guidelines.
  • I documented the Java code using JavaDoc style.

Client

  • Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data (e.g. using paging).
  • I strictly followed the principle of data economy for all client-server REST calls.
  • I strictly followed the client coding and design guidelines.
  • I documented the React code.

Motivation and Context

This pull request introduces a new, detailed contribution analysis pipeline for repositories, allowing us to compute and store granular per-student contribution metrics. Alongside this, the API endpoint is updated to return simplified, aggregated data to the client, improving efficiency and relevance.

Description

Contribution Analysis Pipeline

  • Granular Git Analysis: Extracts lines added & deleted per commit using Git history and VCS logs, and records commit count per student.
  • Database Integration: The extracted data from Artemis and contribution statistics are now persisted in the database, allowing reuse.
    • Adds new database entities and repositories specifically for storing and retrieving student contribution data.
    • Includes necessary Liquibase changelogs to set up the new database structure.

API and Data Model Changes

  • Data Simplification: Added new, simplified DTOs that focus on delivering the aggregated team and student analysis data required by the client, significantly reducing and simplifying the data sent over the wire.
  • Adjusted Endpoint: The data fetching logic is consolidated and exposed through an adjusted endpoint that triggers the full fetch, analysis, and response workflow.

Web Layer and Client Updates

  • Client Code Adjustment: The client-side code has been updated to correctly consume and display the data from the new DTOs, including showing the real contribution metrics mentioned earlier. Also the Team List overview page now correctly displays the total commit count and total lines added aggregated per team, based on the new analysis data.
  • Tutor Name Display: Added functionality to display the tutor's name on the client interface.
  • Local Development Proxy: Configured a development proxy in to forward /api requests to the local server, which resolves CORS conflicts that arise when the client and server run on different local ports.

Review Progress

Code Review

  • Code Review 1

Manual Tests

  • Test 1

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 4, 2025

🤖 No OpenAPI or client changes needed.

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 4, 2025

🤖 No OpenAPI or client changes needed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 4, 2025

🤖 OpenAPI spec and client code auto-updated and committed.

@Cathy0123456789 Cathy0123456789 self-assigned this Dec 4, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 4, 2025

🤖 No OpenAPI or client changes needed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 4, 2025

🤖 OpenAPI spec and client code auto-updated and committed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 4, 2025

🤖 No OpenAPI or client changes needed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 4, 2025

🤖 No OpenAPI or client changes needed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 4, 2025

🤖 No OpenAPI or client changes needed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 5, 2025

🤖 No OpenAPI or client changes needed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 5, 2025

🤖 OpenAPI spec and client code auto-updated and committed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 5, 2025

🤖 No OpenAPI or client changes needed.

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 5, 2025

🤖 No OpenAPI or client changes needed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

🤖 No OpenAPI or client changes needed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

🤖 No OpenAPI or client changes needed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

🤖 No OpenAPI or client changes needed.

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

🤖 No OpenAPI or client changes needed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

🤖 No OpenAPI or client changes needed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

🤖 No OpenAPI or client changes needed.

@az108 az108 marked this pull request as ready for review December 9, 2025 16:40
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 9, 2025

🤖 No OpenAPI or client changes needed.

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 9, 2025

🤖 No OpenAPI or client changes needed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 9, 2025

🤖 No OpenAPI or client changes needed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 9, 2025

🤖 OpenAPI spec and client code auto-updated and committed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 9, 2025

🤖 No OpenAPI or client changes needed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 9, 2025

🤖 No OpenAPI or client changes needed.

@az108 az108 merged commit 2329745 into main Dec 10, 2025
15 checks passed
@az108 az108 deleted the feature/cqi-data branch December 10, 2025 02:14
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.

2 participants