Softcatala / web-2015

Repositori per a la versió 3 de la web de Softcatalà
1 stars 3 forks source link

Incloure definicions al conjugador de verbs #238

Closed jordimas closed 3 years ago

jordimas commented 3 years ago

Canvis que cal fer:

Mostrar definició

El conjugador de verbs ara retorna:

Mostra crèdits Sempre es retorna un camp "definition_credits" que hi ha que mostrar al final de la pàgina per donar crèdit i convidar a la participació

URL de l'API Canviar la URL de l'API de https://www.softcatala.org/api/conjugador/search/cantar a https://api.softcatala.org/conjugador/v1/search/cantar

xavivars commented 3 years ago

El text dels crèdits no té sentit quan no hi ha cap definició. Segurament el missatge hauria de ser diferent si hi ha definició a mostrar o només enllaç.

jordimas commented 3 years ago

Ara mateix la lògica que hi ha és aquesta:

https://github.com/Softcatala/conjugador/blob/a2a4c29676d9f91ef9e1643c8177210d159b01d0/extractor/extract.py#L341

Possar el text a crèdits sempre va ser un suggeriment d'en @pereorga per fomentar la participació al diccionari upstream quan no hi ha un terme. És pot millorar la implementació, però la idea em sembla bona.

xavivars commented 3 years ago

Si. Bàsicament, la meua proposta és canviar el text segons si hi ha o no definició (i és independent de la integració a la web)

jordimas commented 3 years ago

Per mi endavant. Jo no estaré davant d'un PC en els pròxims dies, llavors tira milles 👍

lequims commented 3 years ago

L'API ja està canviada doncs la podem configurar amb una opció del WordPress.

xavivars commented 3 years ago

Miquel, no sé què vols dir....

La meua proposata és que si l'API sempre torna el text de crèdits, que en torne un de diferent. I després al WordPress simplement mostra el que li diga l'API

lequims commented 3 years ago

Jo només em referia al canvi d'URL de l'API.

xavivars commented 3 years ago

Ja està a producció