fix: Remove the Approved From and Approved to search field #137

Merged

shad0w merged 2 commits from claude/issue-136 into master

2026-06-25 22:25:10 +00:00

shad0w commented

2026-06-25 21:53:19 +00:00

Owner

Automated fix by Claude Code for #136.

Closes #136

Review, merge, then create a release tag (vX.Y.Z) to deploy.

Automated fix by Claude Code for #136. Closes #136 Review, merge, then create a release tag (vX.Y.Z) to deploy.

shad0w added 1 commit 2026-06-25 21:53:20 +00:00

feat(history): remove Approved From/To search fields

PR checks / checks (pull_request) Failing after 3s

Details

PR checks / integration (pull_request) Successful in 33s

Details

5218eb3717

Drop the two approval-date pickers from the PO History filter bar per
the manager's request — they were deemed unnecessary. Removes the
inputs, their useState hooks, and their entries in buildParams/clear/
hasFilters.

The approvedFrom/approvedTo URL params are left intact server-side
(page.tsx, lib/history-filter.ts, export route) so existing deep-links
from the dashboard "Approved This Month" card and the report drill-downs
keep pre-filtering History by approval date.

Fixes #136

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

shad0w referenced this pull request

2026-06-25 21:53:20 +00:00

[Issue]: Remove the Approved From and Approved to search field #136

shad0w referenced this pull request from a commit

2026-06-25 22:16:06 +00:00

test(history): cover removal of Approved From/To filters (#136)

shad0w added 1 commit 2026-06-25 22:16:06 +00:00

test(history): cover removal of Approved From/To filters (#136 )

PR checks / checks (pull_request) Successful in 51s

Details

PR checks / integration (pull_request) Successful in 32s

Details

efa9d90ddb

Adds a unit test for HistoryFilters asserting the Approved From / Approved
To date filters no longer render (the issue #136 change) while the remaining
filters (created-date range, cost centre, accounting code, status) stay.
Satisfies the test-presence policy that PR #137 was missing.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

shad0w referenced this pull request from a commit

2026-06-25 22:18:03 +00:00

fix(automation): issue watcher must not open PRs without tests

shad0w referenced this pull request

2026-06-25 22:23:34 +00:00

fix(automation): issue watcher must not open PRs without tests #138

shad0w merged commit bf7ea1a9e6 into master

2026-06-25 22:25:10 +00:00

shad0w referenced this pull request from a commit

2026-06-25 22:25:11 +00:00

Merge pull request 'fix: Remove the Approved From and Approved to search field' (#137) from claude/issue-136 into master