Invoice Detail View #697

Merged

tobru merged 14 commits from feat/invoice-detail into main

2026-09-24 09:39:19 +00:00

tobru commented

2026-09-24 09:35:57 +00:00

Owner

No description provided.

tobru added 14 commits

2026-09-24 09:35:57 +00:00

refactor(invoices): extract get_invoice_portal_url 7f6d753975

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

feat(invoices): load a single invoice with lines through its sale order 8d36b8ab8c

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

feat(invoices): fetch the invoice PDF from Odoo for the portal dd2775239f

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

fix(invoices): harden invoice PDF lookup after review 848bcac66a

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

feat(invoices): invoice detail page inside the portal cb503ff8f4

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

feat(invoices): download the invoice PDF through the portal be9d523eb4

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

feat(invoices): link the invoice list to the portal detail page dc56777d84

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

fix(invoices): align pay rule and harden invoice templates after review a02a471f2c

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

fix(invoices): final review touch-ups eeaf4e3e81

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

test: block all network access in the test suite bba5beeb4c

Every outside service must be mocked. The existing forbid_real_odoo
fixture only covers OdooClient; plain requests/urllib calls (such as the
invoice PDF fallback) could still leave the machine. Non-unix socket
connects and DNS lookups now raise NetworkAccessBlocked, a BaseException
so that broad `except Exception` clauses cannot swallow it.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

docs: invoice detail view design and implementation plan 8eddaf87f8

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

fix(invoices): give the invoice PDF render its own 30 s timeout f1618333fb

Odoo needs about 10 s to render an invoice PDF (measured on preprod,
where the filestore is missing so every download renders). The shared
10 s ODOO_TIMEOUT cut the request off right before the answer. The PDF
fetch now uses ODOO_PDF_TIMEOUT (30 s, below gunicorn's 60 s).

The error now names the exception type (still without the tokenised
URL), and the log tells "no stored PDF" apart from "stored PDF has no
content", which is what preprod shows.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

fix(invoices): show the tax label Odoo prints on invoices a64c423545

Use the tax's "Label on Invoices" (description), falling back to its
name, like Odoo's own invoice report: "8.1%" instead of "8.1% Sales".

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

feat(invoices): dashboard header, grouped sections and instance links

Tests / test (push) Successful in 2m15s

Details

8d0da686ef

- Overview card with the total, payment state, an overdue marker and
  the invoice facts next to it.
- Line items grouped per Odoo section, each with its own subtotal.
- Subtotal, taxes, total and amount due in a separate summary block.
- Section titles naming one of the organization's instances (the
  billing annotation's itemGroupDescription) link to that instance.
- Page styles live in css/invoice.css.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

tobru merged commit 498e06cb54 into main

2026-09-24 09:39:19 +00:00

tobru deleted branch feat/invoice-detail

2026-09-24 09:39:19 +00:00

tobru referenced this pull request from a commit

2026-09-24 09:39:20 +00:00

Merge pull request 'Invoice Detail View' (#697) from feat/invoice-detail into main