feat: add (simple) search functionality to app #49
Conversation
not full text search, crdb does not yet support it :(
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
| deletedAt: null, | ||
| OR: [ | ||
| { title: { contains: query } }, | ||
| { content: { contains: query } }, |
There was a problem hiding this comment.
is this case sensitive?
There was a problem hiding this comment.
not sure if there's a need to concat the two columns before searching so that the search terms can span both title and body
There was a problem hiding this comment.
Good point, will make case insensitive.
Not sure what you mean by concat columns. now just querying both columns
There was a problem hiding this comment.
@karrui example:
title: hello world
content: dog mouse cat
searching "hello world dog" will fail since neither the title nor content contains these 3 words
so something like CONCAT(title, ' ', content) ~ "%dog%mouse%cat%" OR CONCAT(content,' ', title) ~ "%dog%mouse%cat%"
There was a problem hiding this comment.
i see. seems like edge case with no real need to support (since we don't even have good full text search 😢) will ignore for now
|
Will keep this branch hanging for now to serve as example for a new feature |
Using simple substring search, since CRDB does not have full text search functionality yet.
Only minimal frontend work since still waiting on design.
TODO: Still need to search in comments, but waiting for redoing of the schema to something Tw*tter-like