search

Softcatala / conjugador

conjugador
7 stars 3 forks source link

Add notes support #42

Closed jmontane closed 2 months ago

jmontane commented 9 months ago

Afegeix suport per a notes/comentaris per als verbs. Vegeu #7 Només fa la part de l'API, quedaria mostrar les notes al frontend.

jordimas commented 9 months ago

Ara mateix el codi fa res ja que no s'exporten les notes. Caldrà afegir-ho a: https://github.com/Softcatala/conjugador/blob/master/extractor/extract.py

Trobo que és important executar el codi i veure que funciona, ara mateix no s'ha provat.

Si alguna cosa és molt complicada ho puc fer jo quan tingui temps

jmontane commented 9 months ago

Ostres, ho he provat en local "make generate-data" i els json tenien les notes. Potser em falta cometre algun canvi. Ho reviso. Gràcies.

jmontane commented 9 months ago

Em penso que era això. Faltaven canvis per cometre. .

Ara sí que hi hauria de ser tot. Sorry

jordimas commented 9 months ago

Bona feina Joan, jo ho veig molt bé.

L'única cosa es que no passa el UT: https://gitlab.softcatala.org/github/conjugador/-/pipelines

Sembla que el git no es va sincronitzar quan vas fer els canvis perquè estava caigut. Ho he provat en local i funciona, llavors és un problema de Gitlab, que s'autosolucionarà.

L'única cosa que diria és abans de fer merge comentar-ho al grup de Web de Telegram per veure el tema d'UI i tal.

jmontane commented 9 months ago

Gràcies, he afegit un petit commit, a veure si ajuda que sincronitzi amb el gitlab.