Skip to content

Fix frontend bugs from the backlog (#153) - #164

Merged
MatheusCastro99 merged 5 commits into
masterfrom
fix/153-backlog-bugs
Nov 14, 2026
Merged

MatheusCastro99 merged 5 commits into
masterfrom
fix/153-backlog-bugs

Conversation

@MatheusCastro99

Copy link
Copy Markdown
Owner

Part of #153 (the issue stays open as the backlog). Plan: #153 (comment)

Fixes: saved PDF shows description and lines, leftover console log removed, Edit Invoice no longer flags its own number, explicit $0 subtotal accepted, unreadable tax rate is an error, latest customer pick wins. Roadmap notes added for the renamed-customer and in-place update items.

🤖 Generated with Claude Code

MatheusCastro99 and others added 5 commits November 14, 2026 08:05
…r log (#153)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…wins (#153)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…153)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@MatheusCastro99
MatheusCastro99 marked this pull request as ready for review November 14, 2026 13:19
@MatheusCastro99

Copy link
Copy Markdown
Owner Author

Ready to merge at f06ebad0b1483105831c22005c567c9b5ab47b3c. Reviewed SHA = PR head = the SHA CI passed on.

G2: red-team approved with no blocking findings; the UI checks for sub-phases 1–4 passed in the browser on throwaway data. One low finding (the Taxes row reads "0%" when there is no rate) was declined here and logged on #153 with two adjacent items: #153 (comment)

Not checked in the browser: the downloaded PDF file (same document component as the on-page viewer, which was checked).

The PR does not close #153; after merging, run /close 153.

Copilot AI balanced review requested due to automatic review settings November 14, 2026 13:19

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@MatheusCastro99
MatheusCastro99 merged commit ad514cf into master Nov 14, 2026
8 checks passed
@MatheusCastro99
MatheusCastro99 deleted the fix/153-backlog-bugs branch November 14, 2026 13:20
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