Skip to content

Us13 jobs report#30

Merged
grzechuzz merged 3 commits into
masterfrom
us13-jobs-report
Jun 12, 2025
Merged

Us13 jobs report#30
grzechuzz merged 3 commits into
masterfrom
us13-jobs-report

Conversation

@NN0X

@NN0X NN0X commented Jun 12, 2025

Copy link
Copy Markdown
Owner

No description provided.

@NN0X NN0X requested review from Copilot and grzechuzz June 12, 2025 15:01
@NN0X

NN0X commented Jun 12, 2025

Copy link
Copy Markdown
Owner Author

FIRST MERGE US12 @grzechuzz

This comment was marked as outdated.

NN0X and others added 2 commits June 12, 2025 17:04
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
grzechuzz
grzechuzz previously approved these changes Jun 12, 2025
@NN0X NN0X dismissed grzechuzz’s stale review June 12, 2025 17:59

The merge-base changed after approval.

@grzechuzz grzechuzz self-requested a review June 12, 2025 18:00
grzechuzz
grzechuzz previously approved these changes Jun 12, 2025
@grzechuzz grzechuzz requested a review from Copilot June 12, 2025 18:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 new reporting features for service orders by adding two new views and endpoints to display and export repair cost and monthly repair summary reports.

  • Added Repair Cost Report view and associated PDF export functionality.
  • Added Monthly Repair Summary view and PDF export functionality along with corresponding service methods and documents.
  • Updated navigation views to include links for the new reports.

Reviewed Changes

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

Show a summary per file
File Description
CarWorkshopManager/Views/ServiceOrder/RepairCostReport.cshtml New view for displaying repair cost report.
CarWorkshopManager/Views/ServiceOrder/MonthlyRepairSummary.cshtml New view for monthly repair summary report.
CarWorkshopManager/Views/ServiceOrder/Index.cshtml Updated index view with link to the repair cost report.
CarWorkshopManager/Views/Dashboard/Index.cshtml Added link to monthly repair summary report.
CarWorkshopManager/ViewModels/ServiceOrder/*.cs New view models for both reports.
CarWorkshopManager/Services/* New service methods for fetching report data.
CarWorkshopManager/Documents/*.cs New document classes to generate PDFs using QuestPDF.
CarWorkshopManager/Controllers/ServiceOrderController.cs Added endpoints for rendering views and generating PDFs.
CarWorkshopManager/Program.cs Configured QuestPDF license.
CarWorkshopManager/CarWorkshopManager.csproj Included QuestPDF package reference.

Comment thread CarWorkshopManager/Views/ServiceOrder/Index.cshtml
@NN0X NN0X dismissed grzechuzz’s stale review June 12, 2025 18:05

The merge-base changed after approval.

grzechuzz
grzechuzz previously approved these changes Jun 12, 2025
@NN0X NN0X dismissed grzechuzz’s stale review June 12, 2025 18:17

The merge-base changed after approval.

grzechuzz
grzechuzz previously approved these changes Jun 12, 2025
@NN0X NN0X closed this Jun 12, 2025
@NN0X NN0X reopened this Jun 12, 2025
@NN0X NN0X dismissed grzechuzz’s stale review June 12, 2025 18:21

The merge-base changed after approval.

@NN0X NN0X requested a review from grzechuzz June 12, 2025 18:22
@grzechuzz grzechuzz merged commit 3bdf99c into master Jun 12, 2025
2 checks passed
@grzechuzz grzechuzz deleted the us13-jobs-report branch June 12, 2025 18:25
NN0X pushed a commit that referenced this pull request Jun 13, 2025
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