minilauta / miniboard

Minimalistic oldschool imageboard software
MIT License
7 stars 5 forks source link

Mahdollista catalogin lankojen yksinkertainen suodatus. #66

Closed olioton closed 1 year ago

Anonyymi commented 1 year ago

Oma ajatukseni on ollut, että suodatus toteutetaan backendin puolella query parametreilla. Siinä on monia etuja client-puolen ratkaisuun verrattuna.

olioton commented 1 year ago

Olisihan siitä ainakin se etu että seuraaville sivuille menevät langat voitaisiin myös ottaa huomioon. Voisin katsoa jos saisin siirrettyä suodatuksen palvelinpuolella. Onko siitä muita hyötyjä jotka on hyvä ottaa huomioon kuin se, että saa kaikki täsmäävät langat esille? Jos tietokannasta aletaan etsimään alimerkkijonoja, niin se tietty voi olla vähän epätehokasta, mutta tuskin ongelma ellei lankoja ole hyvin paljon?

Anonyymi commented 1 year ago

En usko, että suorituskyky on ongelma. Backend suodatuksen etu on, että se ei ole riippuvainen näkymästä. Eli kun sen kerran toteuttaisi niin se sama koodi toimisi jokaisessa näkymässä jossa listataan lankoja tai postauksia. Ja toinen etu toki tuo, jonka mainitsit.

Suodatus on luultavasti aika yksinkertainen lisä nykyiseen API:in, mutta mielestäni sitä kannattaisi issue puolella speksata ennen kuin joku alkaa tekemään. Sama juttu vähän joka asiassa, en halua että kukaan tekee turhaa työtä, eli suunnitellaan mieluusti ennen toteutusta.

olioton commented 1 year ago

Ehdin jo koittaa tuollaista toteutusta aikaisemmin.

Anonyymi commented 1 year ago

Ei oteta tätä vielä ainakaan mukaan, mielestäni toiminto vaatii enemmän pohdiskelua.