Skip to content

Dependency audit and customer pagination (#168) - #174

Merged
MatheusCastro99 merged 4 commits into
masterfrom
chore/168-dependency-audit-customer-pagination
Nov 14, 2026
Merged

Dependency audit and customer pagination (#168)#174
MatheusCastro99 merged 4 commits into
masterfrom
chore/168-dependency-audit-customer-pagination

Conversation

@MatheusCastro99

@MatheusCastro99 MatheusCastro99 commented Nov 14, 2026

Copy link
Copy Markdown
Owner

Closes #168. Plan: #168 (comment)

🤖 Generated with Claude Code

MatheusCastro99 and others added 3 commits November 14, 2026 16:36
@MatheusCastro99 @claude
chore: npm audit fix in the root and FRONTEND (#168)
74c2771 
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@MatheusCastro99 @claude
feat: fetch every customer for the invoice picker (#168)
e274f36 
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@MatheusCastro99 @claude
feat: page the home page customer cards, 20 to a page (#168)
1e918c9 
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@MatheusCastro99 MatheusCastro99 mentioned this pull request Nov 14, 2026
Closed
4 tasks
@MatheusCastro99 @claude
fix: keep the focused page blue and reject a malformed customer list (#…
a85bc9c 
…168)

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

MatheusCastro99 commented Nov 14, 2026

Copy link
Copy Markdown
Owner Author

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

  • G2 review at 1e918c9: two non-blocking findings, both fixed in a85bc9c as the owner decided (the keyboard-focused selected page now stays blue at 6.70:1; a malformed customer list answer now shows an error toast instead of "no customers").
  • Narrow re-review at a85bc9c: code re-review approved, UI re-check passed.
  • Notes, no change needed: in the dev server React's StrictMode sends the customer list request twice on load (the same page=1&limit=100 request); a production build was not checked. On the home page, the customer and invoice loads share one loading flag (pre-existing).

🤖 Generated with Claude Code

Copilot AI balanced review requested due to automatic review settings November 14, 2026 22:34
Copilot AI reviewed Nov 14, 2026

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 mentioned this pull request Nov 14, 2026
Open
1 of 11 tasks
@MatheusCastro99
MatheusCastro99 merged commit c152be0 into master Nov 14, 2026
7 checks passed
@MatheusCastro99
MatheusCastro99 deleted the chore/168-dependency-audit-customer-pagination branch November 14, 2026 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Issue 0: dependency audit and customer pagination

2 participants

@MatheusCastro99