Skip to content
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

Q&A agents #598

Open
5 of 17 tasks
Tracked by #606
jonquet opened this issue Aug 5, 2024 · 0 comments
Open
5 of 17 tasks
Tracked by #606

Q&A agents #598

jonquet opened this issue Aug 5, 2024 · 0 comments
Assignees

Comments

@jonquet
Copy link
Contributor

jonquet commented Aug 5, 2024

Qq commentaires et améliorations sur les agents avec les changement de la release v2.9.1

  • 1 - Lors qu'on ouvre un agent dans la page d'edition, avec le bouton "stylo" le seul moyen de fermer le formulaire ouvert est de faire "Save" et donc de renvoyer un message au serveur meme si on a rien changé/édité. L'amélioration consisterait a mettre un bouton "Close" a coté de Save pour fermer le formulaire, sans re-soliciter le serveur. C'est d'apre smoi ce qui créera le moins de conflit avec les autres occurences de cette UI e.g. à partir de la page admin (en modal).
    Dans la page edition
Capture d’écran 2024-08-05 à 15 47 31 Dans la page admin Capture d’écran 2024-08-05 à 16 11 11 Capture d’écran 2024-08-05 à 15 50 46
  • 3 - Maintenant qu'on a "re-inforce" l'usage des ORCID et des ROR ... ceux-ci étant des identifiants, en théorie il faudrait que quand on édite un agent on ne puisse pas ajouter d'autres identifiants (au moins via l'UI). Dans le back, on peut rien changer car on a le paramêtre "schemaAgency" qui permet de distinguer les identifiants.
Capture d’écran 2024-08-05 à 15 54 43 Side effect: cela va alléger l'UI aussi quand on édite un agent.
  • 4 - Suis pas sur de l'état de cela sur StagePortal actuellement, mais pour le moment je peux saisir un ORCID pour un agent qui soit n'importe quoi. Avec la discussion sur les expressions régulières etc, ca surprends quand meme que le front check pas un minimum avant s'aller enregistrer ca dans la base.
Capture d’écran 2024-08-05 à 15 58 19

A y re-reflechir, j'aurai peut etre juste enlever du code les expressions régulieres de flexibilité (i.e. qui essayent de trouver la sequence de l'ORCID dans ce qui est mis dans le champ) et remplacer ca par un simple ORCID validator regular expression (unique) qui vérifie que le champ respecte ca et alerte (avant meme de clicker sur save) sur le fait que ce champ n'est pas valide.
En théorie, j'aurai vu ca comme un composant texte (dans le lookbook) qui passe en rouge tant que le texte dedans ne respecte pas une expression réguliere donnée. Pas plus. On a se composant à disposition dans l'UI et à chaque fois qu'on veut faire un check front.. on peut l'utilsier. E.g., piru l'ORCID cela peut etre le meme sur la page d'édition d'un utilisateur. On devrait en fait avoir un sous type de Input::TextInputComponent dans le lookbook avec comme attributs de plus: un mini icon qu'on met a a gauche du texte et une expression régulier de validation.

  • 5 - La requête https://data.stageportal.lirmm.fr/search/agents?query=INRAE nous renvoie en 4eme position l'INIST qui dans sa description ne contient pas du tout la chaine de caractère "INRAE"... ni la sous chaine "INRA". Pourtant les 5eme à 8 eme positon contienne cette sous chaine mais sont classés après.

  • 6 - Dans la recherche: https://data.stageportal.lirmm.fr/search/agents?query=INRAE => Les id, resource_id et creator_t sont préfixées avec le mauvais name space (bioontology.org).

  • 7 - Dans la recherche d'agents via le nouveau endpoint "spécialisé" pourquoi les propriétés JSON ne sont pas les memes que pour les agents eux memes ? En fait par extension, les resultats des "nouveaux" endoints devraient être similaire a ceux du /search endpoint.. entre autre utiliser une description JSON-LD de ce qu'ils renvoient. Par exemple, pour ce resultat: https://data.stageportal.lirmm.fr/search/agents?query=jonquet

Capture d’écran 2024-08-05 à 16 28 44

Il s'agirait de renvoyer une collection avec comme elements, une version plus light de la description JSON-LD d'un agent:
Capture d’écran 2024-08-05 à 16 30 17

Capture d’écran 2024-08-05 à 16 26 24
  • 9 - Une chance qu'on puisse faire aussi : Ne pas pouvoir supprimer un agent de type organization a mons d'etre sur qu'il n'est pas utilisé comme affiliation d'un autre agent #544 ? cela permettrait de fermer tous les issues ouvertes sur les agent sil me semble: Agents issues #523

  • 10 - Dans un soucis d'harmonisation des UI, la table des agents dans la page admin pourrez utiliser l'icone de poubelle dans la colonne Actions pluot que le mot "delete". Egalement, de la place a droite de "Delete" peut etre recuprée (espace vide) pour le repartir dans d'autres colonnes.

  • 11 - Le classement par la colonne "Usages" de la table des agents dans la page admin ne fonctionne pas.

  • 12 - La colonne "Identifiers" de la table des agents dans la page admin pourrait juste afficher le logo ORCID ou ROR quand un identifiant est la et faire en sorte que ce lien soit clikcable vers l'extérieur. Cela économiserait pas mal de largeur dans cette table et permettrait de rendre la colonne name plus large (acronym aussi un peu car pour le moment le triangle de tri par cette colonne apparait au dessus du m).

  • 13 - La colonne "First name" de la table des agents dans la page admin doit être renommée "Name"

  • 14 - Le bouton d'affichage des organizations devrait s'appuyer sur l'acronyme plutôt que le name de l'agent. Cela permettrait le plus souvent d'éviter les 3 points de suspensions (mettre un fallback si pas d'acronyme alors utiliser le name).

Capture d’écran 2024-08-05 à 16 54 00
  • 15 - Le tooltip d'affichage des organizations devrait afficher l'acronyme quand présent à la fin de la chaine de caractère du nom en utilsiant des parenthèses. Ici: '... Environment (INRAE)"
Capture d’écran 2024-08-05 à 16 50 20
  • 16 - Dans le tooltip d'affichage des agents (pas celui des contacts) il ne faut pas afficher l'email. Nous avions vu que comme c'est une information personnelle, on ne l'affiche pas. En plus, ce tooltip utilise 3 lignes, et l'email qui arrive avant l'organization de l'agent empêche de voir l'organization.
Capture d’écran 2024-08-05 à 16 59 47
  • 17 - Bug de changement de type: Sur StagePortal, dans l'edition d'une ontologie, sélectionner "Clément Jonquet" dans un champ agent. Ensuite, ajouter l'affiliation "MISTEA" a Clement en l'éditant.... voir que MISTEA n'est pas de bon type (person au lieu de organisation) alors éditer l'agent MISTEA a l'interieur du formulaire de l'edition de l'agent Clement pour changer son type vers "organisation" => cela ferme le 2eme formulaire et affecte le type "organisation" a Clement au lieu de l'affecter a MISTEA
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants