Improve taxon search box #557
Conversation
jpm-cbna
force-pushed
the
feat/taxons-search
branch
from
787a279
to
519b824
Compare
| -- ISSUE #531 & #532 | ||
| CREATE EXTENSION IF NOT EXISTS unaccent SCHEMA "public"; | ||
|
|
||
| -- ISSUE #532 |
There was a problem hiding this comment.
Il est également nécessaire de donner les droits à l'utilisateur en lecture seule de l'Atlas (habituellement geonatatlas) :
GRANT SELECT ON TABLE atlas.vm_search_taxon TO geonatatlas;Vue que le nom de cet utilisateur est configurable et que l'accès au paramètre de config n'est pas possible, comment doit-on procéder ?
There was a problem hiding this comment.
Je mettrais une note spécifique dans le changelog, indiquant la commande à lancer manuellement
There was a problem hiding this comment.
Je mettrais une note spécifique dans le changelog, indiquant la commande à lancer manuellement
Tu veux dire que je peux modifier le fichier https://github.com/PnX-SI/GeoNature-atlas/blob/master/docs/changelog.rst ? Ou c'est à faire uniquement lors de la création de la nouvelle release ?
There was a problem hiding this comment.
Oui c'est bien de compléter le Changelog au fur et à mesure dans les PR, ça rend les merge un peu plus complexe, mais on s'en charge.
jpm-cbna
force-pushed
the
feat/taxons-search
branch
from
519b824
to
b65f025
Compare
jpm-cbna
force-pushed
the
feat/taxons-search
branch
from
b65f025
to
286fa35
Compare
| cd_nom, | ||
| lb_nom, | ||
| cd_ref, | ||
| STRING_TO_TABLE(nom_vern, ', ') AS nom_vern |
There was a problem hiding this comment.
Avec SRING_TO_TABLE on ne pourra plus rechercher sur tous les noms vernaculaires, je pense que c'est dommage. Il me semble que que le split n'était fait que sur la fiche espèce, pour de l'affichage
There was a problem hiding this comment.
oups j'avais pas vu la différence entre search_name et display_name
There was a problem hiding this comment.
Normalement, c'est fonctionnel. J'ai mis en prod cette PR sur les 2 Atlas des SINP PACA et AURA, si tu veux vérifier certains points : https://nature.silene.eu/ et https://atlas.biodiversite-auvergne-rhone-alpes.fr/
Est ce que tu penses que j'ai loupé quelque chose ?
There was a problem hiding this comment.
C'est top, ça améliore bien la recherche je trouve
micro remarque de forme : tu mets que le nom de l'espece en gras. Sur l'INPN ou le site de taxref, le nom plus l'autorité sont en gras. A moins que ce soit vraiment un souhait, je me dis autant faire comme eux ? En plus ça évite le "replace". On englobe juste le "nom_complet" de balise <b> ?
Alors je suppose que cela dépends où sur l'INPN car par exemple la fenêtre des résultats de recherche ne contient pas l'autorité en gras: Tu pourrais m'indiquer les URL des pages que tu mentionnes ? En fait, j'ai évité de mettre l'autorité en gras pour ne pas surcharger la lecture du nom. Je trouve que c'est plus lisible et ergonomique ainsi... Le "replace" est aussi très rapide et ne semble pas poser de problème de performance. |
|
C'était ici : https://taxref.mnhn.fr/taxref-web/taxa/search et sur la page d'accueil de l'INPN : |
Ok, effectivement. J'ai essayé de voir s'il existait des recommandations sur le sujet mais je n'ai rien trouvé. Du coup, on fait comme tu veux. Personnellement, je trouve plus lisible de ne pas mettre en gras l'autorité... |
|
ok ça marche ! |
See: #532