Skip to content

Conversation

@GV14982
Copy link
Contributor

@GV14982 GV14982 commented Oct 13, 2025

  • Added tags field on Feed struct
  • Added filter shorthand for tags
  • Currently filtering doesn't work :sad:

@GV14982
Copy link
Contributor Author

GV14982 commented Oct 13, 2025

@guyfedwards Alright I feel kinda dumb... I have no idea how the filtering works.

I think my issue is that the tags don't exist in the text that is displayed, unlike the feed name. So when I try to filter by a tag, I just get no results 🙃 Any suggestions for how to achieve this filtering?

@guyfedwards
Copy link
Owner

No worries, I'm not super on top of the filtering stuff myself.
My initial simple solution would be to add an additional searchTerm field or something to the struct that can be used for filtering and is the title/name/tags/whatever else concatenated together.
Does that make sense?

- Added tags field on `Feed` struct
- Added filter shorthand for tags
- Currently filtering doesn't work :sad:
@GV14982 GV14982 force-pushed the feat/add-tag-feature branch from f69418f to 6ca4e70 Compare October 14, 2025 20:24
@GV14982 GV14982 marked this pull request as ready for review October 14, 2025 21:44
@GV14982
Copy link
Contributor Author

GV14982 commented Oct 15, 2025

Oops, forgot to mention this closes #149

@guyfedwards
Copy link
Owner

@GV14982 branch will need an update after the other changes.

@GV14982
Copy link
Contributor Author

GV14982 commented Oct 18, 2025

@guyfedwards updated.

@guyfedwards guyfedwards merged commit a50fd43 into guyfedwards:master Oct 18, 2025
2 checks passed
@guyfedwards
Copy link
Owner

thanks @GV14982

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants