-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve taxon search box #557
base: develop
Are you sure you want to change the base?
Conversation
787a279
to
519b824
Compare
-- ISSUE #531 & #532 | ||
CREATE EXTENSION IF NOT EXISTS unaccent SCHEMA "public"; | ||
|
||
-- ISSUE #532 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je mettrais une note spécifique dans le changelog, indiquant la commande à lancer manuellement
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
519b824
to
b65f025
Compare
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oups j'avais pas vu la différence entre search_name
et display_name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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>
?
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