Skip to content

Add filter by stream title#81

Merged
mnsinri merged 3 commits into
mnsinri:mainfrom
neuenmuller:add-filter-by-stream-title
Dec 7, 2025
Merged

Add filter by stream title#81
mnsinri merged 3 commits into
mnsinri:mainfrom
neuenmuller:add-filter-by-stream-title

Conversation

@sarisia
Copy link
Copy Markdown
Contributor

@sarisia sarisia commented Dec 7, 2025

This pull request adds support for filtering streams by title, allowing users to input a title filter that updates the displayed streams accordingly. It introduces a reusable Input component and integrates it into the header UI for filtering. The settings schema and context are updated to handle the new filter state, and the stream provider now applies the filter logic when displaying streams.

Stream Filtering Feature:

  • Added a new filteredTitle property to the settings schema (SettingSchema in src/providers/setting/schema.ts) and updated action types to support it. [1] [2]
  • Updated the stream provider to filter streams by title using the new filteredTitle value, ensuring only streams matching the filter are shown. [1] [2] [3]

UI Enhancements:

  • Created a reusable Input component in src/components/ui/input.tsx for consistent input styling and behavior.
  • Integrated the Input component into the header, allowing users to filter streams by title directly from the UI. [1] [2] [3]

State Management:

  • Updated the header view model to manage the title filter state and dispatch changes to the settings context. [1] [2]

Copilot AI and others added 3 commits December 7, 2025 11:00
Co-authored-by: sarisia <33576079+sarisia@users.noreply.github.com>
Co-authored-by: sarisia <33576079+sarisia@users.noreply.github.com>
@sarisia
Copy link
Copy Markdown
Contributor Author

sarisia commented Dec 7, 2025

@mnsinri sorry to ping you but since this project relies on Firebase heavily, local preview is not working. Could you please provide preview deployment? Thanks!

@mnsinri mnsinri merged commit 43c4f65 into mnsinri:main Dec 7, 2025
2 checks passed
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