Skip to content

feat: add service type and category filters to search and services pages - #527

Open
phrogwrld wants to merge 4 commits into
mainfrom
feat/service-search-filters
Open

phrogwrld wants to merge 4 commits into
mainfrom
feat/service-search-filters

Conversation

@phrogwrld

@phrogwrld phrogwrld commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

Description

Add filtering by service type (digital/information) and category to the search results and services pages. Users can select filters via checkbo dropdown s, see the active filters, and

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Changes Made

Notes

Testing

  • Visual regression tests added for all device sizes
  • Verify all pages render correctly
  • Test responsive layout across device sizes (snapshots created)
  • Check accessibility standards are met
  • Validate markdown formatting renders properly
  • Test navigation and breadcrumbs

Related Github Issue(s)/Trello Ticket(s)

https://trello.com/c/sIHRyjPV

Checklist

  • Code follows project style guidelines
  • Self-review completed
  • Tests added/updated (visual regression snapshots)
  • Documentation updated

@amazon-inspector-n-virginia

Copy link
Copy Markdown

⏳ I'm reviewing this pull request for security vulnerabilities and code quality issues. I'll provide an update when I'm done

@vercel

vercel Bot commented Apr 16, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
frontend-alpha Error Error Apr 22, 2026 1:25pm

Request Review

@amazon-inspector-n-virginia

Copy link
Copy Markdown

✅ I finished the code review, and didn't find any security or code quality issues.

@sajclarke sajclarke left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

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.

3 participants