Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Search v1 #19

Merged
merged 4 commits into from
Feb 13, 2024
Merged

Search v1 #19

merged 4 commits into from
Feb 13, 2024

Conversation

kieraneglin
Copy link
Owner

What's new?

  • Adds a media search method that uses postgres' fulltext search to look at media titles and descriptions
  • Adds a description field to media_items and populates it on index/download
  • Adds a truncate method to StringUtils for cutting text to length and maybe adding ...
  • Adds UI to render a search page including highlighting the relevant excerpts for each search result

What's changed?

  • Updated header template so that the search form actually works
  • Updated length(list) > 0 to match?([_|_], list) in a few places. It's a little faster, but mainly it's more elixir-y imo
  • Updated header styles to show/hide appropriate elements on all viewport sizes

What's fixed?

N/A

Any other comments?

This is a very basic implementation of search, but it works well enough for now. Like so many other things, the objective is to get it out the door and I'll iterate on the aspects of the app that are lacking

@kieraneglin kieraneglin self-assigned this Feb 13, 2024
@kieraneglin kieraneglin merged commit 060e340 into master Feb 13, 2024
1 check passed
@kieraneglin kieraneglin deleted the ke/search-v1 branch February 13, 2024 22:46
kieraneglin added a commit that referenced this pull request Mar 29, 2024
* Adds description to media items; hooks it up to indexing/media downloading

* Added a search method using postgres fulltext search

* Hooked up search functionality to the search form

* Added persistence to the search form when on search page
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.

1 participant