search

Mikael-Lenander / vihreiden-ohjelma-alusta

Vihreät ohjelmat-sivusto. Pohjana atomic server
https://atomicserver.eu
MIT License
0 stars 0 forks source link

Add ActionLists for managing ActionItems #33

Closed jannepeltola closed 2 weeks ago

jannepeltola commented 1 month ago

Tässä olisi ensimmäinen versio #12 ratkaisuksi. Vaikutti hyvältä ensimmäiseltä full-stack-ongelmalta.

Muutokset luovat uuden ActionList-luokan ontologiaan. Luokka sisältää vanhoja ActionItemejä. Tätä varten on muokattu ohjelmien rendausta, ontologiaa sekä ohjelmien Markdown-parseria.

Parseriin tuli tässä hieman tilanhallintaa, jota tein vähän geneerisemmäksi append-muuttujalla, jolloin voidaan jatkossa lisätä lisää vastaavaa kerryttävää logiikkaa. Jos oletetaan pelkät listat, tilan käyttöä vähentäisi if this_elements is None.

Testattu lokaalisti.

lippinj commented 4 weeks ago

Tämä toki toimii, mietin vain, onko viisasta peilata näitä rakennetta tietokannassa (vs. parsia rakenne renderöintivaiheessa). Toki tällä tavalla se menee helpoiten (erityisesti koska se mahdollistaa elementsien ja react-kikkareiden käsittelemisen 1-to-1). Oliko sulla joku näkemys tähän?

jannepeltola commented 2 weeks ago

Tämä toki toimii, mietin vain, onko viisasta peilata näitä rakennetta tietokannassa (vs. parsia rakenne renderöintivaiheessa). Toki tällä tavalla se menee helpoiten (erityisesti koska se mahdollistaa elementsien ja react-kikkareiden käsittelemisen 1-to-1). Oliko sulla joku näkemys tähän?

Tämä oli juurikin tavoitteena, että saadaan yksi toimenpide yhdeksi olioksi. Jatkokehityksen kannalta järkevämpää. Voisin jumppailla tästä uuden PR:n vielä uuteen ympäristöön ja ottaa tuon import-bugin huomioon...