Skip to content

web: match board search against visible tags as well as titles - #71

Open
nahuel007ng wants to merge 1 commit into
joelborch:mainfrom
nahuel007ng:web/board-search-tags
Open

nahuel007ng wants to merge 1 commit into
joelborch:mainfrom
nahuel007ng:web/board-search-tags

Conversation

@nahuel007ng

Copy link
Copy Markdown

What

Closes #65

cardFilter in apps/web/src/board/filters.ts now keeps a card when the trimmed, lowercased search query is a substring of its title or of any visible tag. Before, it only checked the title. Machine-only ext: tags never match, because the tags go through visibleTags from apps/web/src/lib/fmt.ts, the same helper the card chrome uses. The assignee and "mine only" filters combine with search as before, and filtersActive is unchanged.

The new apps/web/src/board/filters.test.ts covers a title match, a tag match, an ext: tag that does not match, and search combined with an assignee filter.

Gates run locally with Node 22 and pnpm 11.12.0: pnpm typecheck passes, pnpm -r test passes (web 112, core 415, api 476, importer 63), and pnpm --filter @flow/web build succeeds.

Checklist

  • pnpm typecheck and pnpm -r test pass locally
  • No secrets or real config values (account IDs, hostnames, tokens, wrangler.jsonc) are included — only wrangler.example.jsonc placeholders
  • Changes to the API contract (routes, @flow/shared schemas, DO storage shape, MCP tools) are called out below
  • Docs / READMEs updated if behavior or setup changed (no docs describe the search box)

Contract changes

None. Client-side filter only; tasks in the board snapshot already carry tags.

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.

Board search should match tags as well as titles

1 participant