[Quotes]: Download the PDF for a quote from the quote list

[nielsdrost7]

Context

Downloading a quote PDF currently requires opening the quote edit view and finding the export option. A one-click PDF download from the quotes list makes it faster to retrieve PDFs without navigating away.

Related: the same PDF generation service used by invoice download (#200) should be reusable here.

---

User story

As a user, I want to click Download PDF in the quote row dropdown so I can get the PDF without opening the full edit form.

---

Acceptance criteria

• A Download PDF action exists in the quote row action group.
• Clicking it triggers a PDF download of that quote.
• The action is visible for all quote statuses.
• If no PDF can be generated, the user sees a meaningful error notification.

---

Arrange · Act · Assert

// Modules/Quotes/Tests/Feature/QuotesTest.php

#[Test]
public function it_downloads_pdf_from_quote_list(): void
{
    /* Arrange */
    $quote = Quote::factory()->for($this->company)->create([
        'quote_number' => 'QUO-2026-001',
    ]);

    /* Act */
    $component = Livewire::actingAs($this->user)
        ->test(ListQuotes::class, ['tenant' => Str::lower($this->company->search_code)])
        ->callTableAction('download-pdf', $quote);

    /* Assert */
    $component->assertSuccessful();
    $component->assertFileDownloaded('QUO-2026-001.pdf');
}

[coderabbitai]

📝 CodeRabbit Plan Mode

Generate an implementation plan and prompts that you can use with your favorite coding agent.

• Create Plan

Examples

• Example 1
• Example 2

---

🔗 Similar Issues

Related Issues

• As a user, viewing all the clients, i want to click on a dropdown and directly **view** that specific client #194
• [Invoices]: Download PDF for an invoice from the invoice view #190
• [Clients]: Create a quote for a specific client from the client list #191
• [Clients]: Create an invoice for a specific client from the client list #192
• As a user, viewing all the clients, i want to click on a dropdown and directly **edit** that specific client #193

👤 Suggested Assignees

• nielsdrost7

---

🧪 Issue enrichment is currently in open beta.

You can configure auto-planning by selecting labels in the issue_enrichment configuration.

To disable automatic issue enrichment, add the following to your .coderabbit.yaml:

issue_enrichment:
  auto_enrich:
    enabled: false

💬 Have feedback or questions? Drop into our discord!

[nielsdrost7]

@coderabbitai make pull-request with AAA test Scenario's: Arrange, Act, Assert. And also make sure you test edge cases

[coderabbitai]

Coding Plan

Approach

Add a dropdown action to the quotes listing table that allows users to download a PDF for a specific quote. The implementation creates a dedicated PDF service leveraging the existing Core PDF infrastructure, integrates it into the Filament table as a row action, and includes comprehensive testing following AAA patterns with edge case coverage.

Observations

The application is a Laravel-based system with a modular architecture using Filament for admin panels. The Quotes module follows a clean separation of concerns with Filament resources, dedicated table configuration classes, and service layer components. The Core module provides a shared PDF generation infrastructure using a factory pattern that supports multiple drivers (dompdf and wkhtmltopdf). Quote templates are stored in custom/templates/quote_templates/ as blade views. The existing HTML helper class in Core demonstrates the pattern for rendering blade templates with model data.

Assumptions

Assumption 1: How should the system determine which template to use for PDF generation?

Options Considered:

• Always use the template stored in the quote's template attribute, falling back to 'custom' if null
• Allow users to select a template at download time through a modal
• Use a system-wide default template configuration

Chosen Option: 1

Rationale: Use the quote's stored template attribute with 'custom' as fallback. This is simpler, aligns with the existing template selection pattern in the Quote model, and doesn't require additional user interaction for a straightforward download action.

Assumption 2: Which PDF driver should be used?

Options Considered:

• Use the driver specified in config('ip.pdfDriver') via PDFFactory
• Hard-code dompdf as the driver
• Allow per-action driver selection

Chosen Option: 1

Rationale: Use PDFFactory with the configured driver. This respects existing system configuration and maintains flexibility for deployment environments that may prefer wkhtmltopdf for better rendering quality.

💡 User Tips

Regenerate the plan with different choices with @coderabbitai <feedback>.

Implementation Steps

Phase 1: PDF Generation Service

Create a dedicated service for generating quote PDFs that leverages the existing Core PDF infrastructure and template system.

Create QuotePDFService

Implement a new service class to handle quote PDF generation and file response creation.

• Create Modules/Quotes/Services/QuotePDFService.php with a generatePdf(Quote $quote) method
• Use PDFFactory::create() to instantiate the configured PDF driver
• Load quote relationships (prospect via Relation model, quoteItems, companyProfile, user) using eager loading
• Determine template path: use $quote->template ?? 'custom' to build path custom/templates/quote_templates/{template}.blade.php
• Render the blade template using view($templatePath)->with('quote', $quote)->render() pattern (following HTML helper pattern)
• Pass rendered HTML to PDF driver's loadHtml() or equivalent method
• Set paper size to A4 and orientation to portrait using driver's configuration methods
• Return a BinaryFileResponse using response()->download() or equivalent with appropriate filename (e.g., quote-{quote_number}.pdf)

🤖 Prompt for AI agents

Create a QuotePDFService that generates PDF files from quote data. The service
should handle the complete PDF generation workflow.

Create the service class at `Modules/Quotes/Services/QuotePDFService.php`:
- Implement a `generatePdf(Quote $quote)` method
- Use PDFFactory to create the configured PDF driver
- Eager load relationships: prospect (via Relation model), quoteItems,
companyProfile, and user
- Determine template path using `$quote->template ?? 'custom'` to construct
`custom/templates/quote_templates/{template}.blade.php`
- Render the blade template with quote data using the view helper
- Pass the rendered HTML to the PDF driver's loadHtml method
- Configure PDF to use A4 paper size and portrait orientation
- Return a BinaryFileResponse with filename pattern `quote-{quote_number}.pdf`

Phase 2: Filament Table Action

Add the dropdown action to the quotes listing table that triggers PDF generation.

Add PDF Download Action to QuotesTable

Integrate the PDF service into the Filament table configuration as a row-level dropdown action.

• Modify Modules/Quotes/Filament/Company/Resources/Quotes/Tables/QuotesTable.php in the actions configuration section
• Add a new Action::make('download_pdf') with appropriate label (use translation key like trans('ip.download_pdf'))
• Set icon to a suitable download icon (e.g., 'heroicon-o-arrow-down-tray')
• In the action callback, inject QuotePDFService and call generatePdf($record) where $record is the Quote instance
• Return the BinaryFileResponse from the service directly
• Place action within an ActionGroup if other row actions exist, or create a dropdown structure

🤖 Prompt for AI agents

Add a PDF download action to the quotes table. This integrates the
QuotePDFService into the Filament table interface.

Modify
`Modules/Quotes/Filament/Company/Resources/Quotes/Tables/QuotesTable.php`:
- Add a new action `Action::make('download_pdf')` to the actions configuration
- Set the label using translation key `trans('ip.download_pdf')`
- Use icon 'heroicon-o-arrow-down-tray'
- In the action callback, inject QuotePDFService and call `generatePdf($record)`
- Return the BinaryFileResponse from the service
- Add to an ActionGroup if other row actions exist, otherwise create a dropdown
structure

Phase 3: Comprehensive Testing

Implement feature and unit tests following AAA pattern with explicit edge case coverage.

Task 1: Create Feature Test for PDF Download Action

Test the Filament action integration and end-to-end PDF download flow.

• Create Modules/Quotes/Tests/Feature/QuotePDFDownloadTest.php extending AbstractCompanyPanelTestCase
• Use explicit /* arrange */, /* act */, /* assert */ comments in each test method
• Test case 1: Successful PDF download
  • Arrange: Create a quote with quote items, relationships (prospect, company profile), and valid template attribute
  • Act: Call the PDF download action using testLivewire()->callAction('download_pdf', record: $quote)
  • Assert: Response is BinaryFileResponse, has correct content type (application/pdf), filename matches pattern
• Test case 2: Quote with null template (fallback to custom)
  • Arrange: Create quote with template set to null
  • Act: Call download action
  • Assert: PDF generates successfully using custom.blade.php
• Test case 3: Quote with missing relationships
  • Arrange: Create quote without eager-loading prospect or quote items
  • Act: Call download action
  • Assert: Service properly loads relationships and generates PDF without errors
• Test case 4: Unauthorized access
  • Arrange: Create quote for a different company
  • Act: Attempt to call download action
  • Assert: Action is not available or fails authorization

Task 2: Create Unit Test for QuotePDFService

Test the PDF service logic in isolation.

• Create Modules/Quotes/Tests/Unit/QuotePDFServiceTest.php extending TestCase with RefreshDatabase trait
• Use explicit AAA comments
• Test case 1: PDF generation with valid quote
  • Arrange: Mock Quote with all relationships, mock PDFFactory to return a driver mock
  • Act: Call generatePdf($quote)
  • Assert: PDFFactory was called, driver received HTML content containing quote data, BinaryFileResponse returned with correct MIME type
• Test case 2: Template resolution logic
  • Arrange: Create quotes with different template values (null, 'custom', hypothetical 'alternate')
  • Act: Call generatePdf
  • Assert: Correct template path is constructed for each case
• Test case 3: Relationship loading
  • Arrange: Create quote without pre-loaded relationships
  • Act: Call generatePdf
  • Assert: Service loads prospect, quoteItems, companyProfile relationships (verify via query count or relationship assertions)
• Test case 4: Blade rendering with quote data
  • Arrange: Create quote with specific values (quote_number, quote_total, prospect name)
  • Act: Generate PDF
  • Assert: Rendered HTML contains expected quote data (use view rendering assertions or string contains checks)

🤖 Prompt for AI agents

Create comprehensive tests for the PDF download functionality following AAA
pattern. This phase creates both feature and unit tests with edge case coverage.

Create feature test `Modules/Quotes/Tests/Feature/QuotePDFDownloadTest.php`:
- Extend AbstractCompanyPanelTestCase
- Use explicit `/* arrange */`, `/* act */`, `/* assert */` comments
- Test successful PDF download with complete quote data
- Test quote with null template (fallback to custom)
- Test quote with missing relationships to verify eager loading
- Test unauthorized access from different company

Create unit test `Modules/Quotes/Tests/Unit/QuotePDFServiceTest.php`:
- Extend TestCase with RefreshDatabase trait
- Use explicit AAA comments
- Test PDF generation with valid quote and mocked dependencies
- Test template resolution logic for different template values (null, 'custom',
'alternate')
- Test relationship loading behavior
- Test blade rendering contains expected quote data

---

🚀 Next Steps

🤖 All AI agent prompts combined

Task: 1

Create a QuotePDFService that generates PDF files from quote data. The service
should handle the complete PDF generation workflow.

Create the service class at `Modules/Quotes/Services/QuotePDFService.php`:
- Implement a `generatePdf(Quote $quote)` method
- Use PDFFactory to create the configured PDF driver
- Eager load relationships: prospect (via Relation model), quoteItems,
companyProfile, and user
- Determine template path using `$quote->template ?? 'custom'` to construct
`custom/templates/quote_templates/{template}.blade.php`
- Render the blade template with quote data using the view helper
- Pass the rendered HTML to the PDF driver's loadHtml method
- Configure PDF to use A4 paper size and portrait orientation
- Return a BinaryFileResponse with filename pattern `quote-{quote_number}.pdf`
===============================================================================

Task: 2

Add a PDF download action to the quotes table. This integrates the
QuotePDFService into the Filament table interface.

Modify
`Modules/Quotes/Filament/Company/Resources/Quotes/Tables/QuotesTable.php`:
- Add a new action `Action::make('download_pdf')` to the actions configuration
- Set the label using translation key `trans('ip.download_pdf')`
- Use icon 'heroicon-o-arrow-down-tray'
- In the action callback, inject QuotePDFService and call `generatePdf($record)`
- Return the BinaryFileResponse from the service
- Add to an ActionGroup if other row actions exist, otherwise create a dropdown
structure
===============================================================================

Task: 3

Create comprehensive tests for the PDF download functionality following AAA
pattern. This phase creates both feature and unit tests with edge case coverage.

Create feature test `Modules/Quotes/Tests/Feature/QuotePDFDownloadTest.php`:
- Extend AbstractCompanyPanelTestCase
- Use explicit `/* arrange */`, `/* act */`, `/* assert */` comments
- Test successful PDF download with complete quote data
- Test quote with null template (fallback to custom)
- Test quote with missing relationships to verify eager loading
- Test unauthorized access from different company

Create unit test `Modules/Quotes/Tests/Unit/QuotePDFServiceTest.php`:
- Extend TestCase with RefreshDatabase trait
- Use explicit AAA comments
- Test PDF generation with valid quote and mocked dependencies
- Test template resolution logic for different template values (null, 'custom',
'alternate')
- Test relationship loading behavior
- Test blade rendering contains expected quote data

💡 Iterate on the plan with: @coderabbitai <feedback>

Example Feedback
- @coderabbitai You can skip phase 3. Add a simple unit test case for phase 2.
- @coderabbitai For assumption 1 go ahead with option 3 and replan.

---

💬 Have feedback or questions? Drop into our discord!