arj03 / ssb-browser-demo

A secure scuttlebutt client interface running in the browser
Other
58 stars 11 forks source link

Add filter for only showing only messages from people you're directly following #50

Closed KyleMaas closed 3 years ago

KyleMaas commented 3 years ago

Adds a filter for showing only messages from people you're directly following. Also makes a few other changes:

arj03 commented 3 years ago

Looks good from the code :) Thanks!

arj03 commented 3 years ago

@KyleMaas was testing this, if you don't follow very many people this could easily return 0 results. It would be better to do this as an author query.

arj03 commented 3 years ago

Other than that I really like the new filter and very nice you cleaned this up.

KyleMaas commented 3 years ago

It would indeed be much better to do it at the database query level, but looking at db2's available operators, I couldn't figure out a way to do it.

You mentioned doing an author query. Do those support supplying a list like an SQL IN? Or would you "or" all of the followed peoples' IDs (possibly resulting in hundreds of "or"s)?

On January 17, 2021 11:08:47 AM EST, Anders Rune Jensen notifications@github.com wrote:

@KyleMaas was testing this, if you don't follow very many people this could easily return 0 results. It would be better to do this as an author query.

-- You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub: https://github.com/arj03/ssb-browser-demo/pull/50#issuecomment-761836204

arj03 commented 3 years ago

@KyleMaas yeah exactly or'ing them. I implemented it here

KyleMaas commented 3 years ago

Perfect. Didn't know you could do that.