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

Major documents update #940

Merged
merged 1 commit into from Feb 8, 2024
Merged

Major documents update #940

merged 1 commit into from Feb 8, 2024

Conversation

vmarseguerra
Copy link
Contributor

@vmarseguerra vmarseguerra commented Jan 14, 2024

The main goal is to improve how to create and view documents.

Most notable changes are:

  • Move the document creation page into the entities creation page
  • Improve document detail page
  • Improve linked entity display in the document detail page
  • Preview known document file

Other important changes:

  • Transform the Document creation form to be a single step
  • Regroup document creation form elements
  • Refactor document creation provider

This PR is linked to the API PR: GrottoCenter/grottocenter-api#1271

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://orange-rock-0d4f87503-940.westeurope.azurestaticapps.net

@urien
Copy link
Contributor

urien commented Jan 14, 2024

Recherche avancée sur les documents :

  • Les champs sont parfaits, il n'est pas possible de faire un test car elasticsearch ne donne pas de résultats. Peut être mettre un commentaire pour préciser les champs : pour Source id : par exemple id originelle des analyses BBS. ISO countries : Apha 2 https://en.wikipedia.org/wiki/ISO_3166-1. ISO Regions voir https://en.wikipedia.org/wiki/ISO_3166-2:XX. XX étant le code pays sur 2 lettres
  • Si C'est possible rallonger la barre permettant de sélectionner la date pour faciliter la sélection des dates récentes
    image

@urien
Copy link
Contributor

urien commented Jan 14, 2024

Visualisation des documents
C'est beaucoup mieux organisé, les pictos pour les documents enfants permettent d'animer un peu les pages
Quelques propositions :

  • Pour les sources c'est pertinent pour le bbs, mais parfois pour faire l'import je suis obligé de mettre des identifiants de mon choix et des noms de type Unknow. On peut laisser ainsi pour l'instant et rechercher une solution pour gérer ces imports de manière globale
  • Pour les sujets BBS il serait intéressant de les considérer comme des chaines traduites de manière a ce qu'elles s'affichent dans la langue de l'utilisateur. Rajouter le code serait un plus sans doute utile pour les personnes qui sont habitués à ce système
  • Il n'y a pas de prévisualisation pour les mp4 https://grottocenter-beta.grottomap.org/ui/documents/147151
  • La vue des images anime bien la page
  • Les entités liées sont bien organisées mais le lien proposé pour l'entrée est une url de cave
  • Si c'est possible il serait utile de faire figurer la licence retenue sur chaque ligne correspondant à un fichier
  • La vue des pdf est vraiment bien
  • Les pages sont un peu tristes, peut être les animer en mettant les détails en valeur : c'est le bloc qui existe toujours même s'il y a très peu d'informations

@urien
Copy link
Contributor

urien commented Jan 14, 2024

Création d'un document
C'est bien d'avoir cet accès unique pour créer des entités puis le choix du type de document avec la référence au BBS et un formulaire en une page.

  • Pour le choix du pays ou de la région il faudrait proposer un lien vers une page avec les codes ISO. J'en ai proposé un, un peu plus haut
  • le choix de la date est explicite et plus condensé
  • la possibilité d'associer le document à un massif a disparu. On avait évoqué deux pb pour ce champ : impossibilité de supprimer un lien a un massif depuis le formulaire et impossibilité de créer un massif inexistant depuis le formulaire, comme c'est possible pour les auteurs et les éditeurs
  • On a un pb depuis quelques mois : quand on créé un éditeur il est bien créé mais il n'est pas affiché comme éditeur sélectionné. Il faut alors faire une recherche et le sélectionner dans la liste qui s'ouvre

@urien
Copy link
Contributor

urien commented Jan 14, 2024

Import CSV
Cela bloque ici
image

pour ce fichier
test issues.csv

@urien
Copy link
Contributor

urien commented Jan 15, 2024

Je ne sais pas si c'est possible mais ce serait bien qu'on puisse aussi lire les mp3., si c'est simple à mettre en place. Il y a plus de 300 interviews de speleos sous licence libre, j'ai archivé le premier https://grottocenter.org/ui/documents/216009

@vmarseguerra
Copy link
Contributor Author

@urien Merci pour ces retours. Normalement ils devraient tous avoir été pris en compte et corrigés.

En ce qui concerne le champ pour associer un massif, il a été supprimer car de la même manière que pour les entrances cette association (et désassociation) peut maintenant être faite sur la page du massif lui-même.

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://orange-rock-0d4f87503-940.westeurope.azurestaticapps.net

@urien
Copy link
Contributor

urien commented Jan 19, 2024

Recherche
Commentaire sur les champs iso countries et iso régions : ok
Rallonger le champ date : ok
Visualisation
champs source iniquement pour le bbs : ok
Prévisualisation pour les mp4 : super
Le lien vers l'entrée au lieu de la cave : ok
Licence : ok
possibilité d'écouter les son : merveilleux !
Création d'un document
Création d'un éditeur ok
PB : pour la création d'un article le choix d'un document parent bloque. De même pour la création d'une issue le choix d'une collection bloque
Le reste fonctionne, la validation et la suppression des documents a valider fonctionnent
Import en masse
J'ai refait le même test avec le même fichier et cela bloque de la même manière

Merci beaucoup pour tout ce travail

@urien
Copy link
Contributor

urien commented Jan 19, 2024

J'ai oublié : ce que tu as fait pour les massifs est vraiment une très belle idée : cela va éviter que les personnes mettent tous les documents concernant des cavités situées dans le massif et plutôt favoriser les documents qui traitent du massif

@vmarseguerra
Copy link
Contributor Author

Le problème le choix d'un document parent devrait être résolu.
Pour l'import CSV je n'arrive pas à le reproduire j'ai bien pu importer le fichier CSV plus haut. Ça bloque toujours au même endroit ?

@urien
Copy link
Contributor

urien commented Jan 22, 2024

Salut @vmarseguerra
Pour un numéro, j'arrive à l'associer à une collection, depuis la page de validation je peux le voir, je peux le valider mais je ne peux pas le modifier
pour les articles, soumettre un autre article fonctionne ! hourra !
Je peux l'associer à son Issue, je peux le visualiser le valider, le décliner mais pas l'éditer

Pour l'import en masse, j'importe le fichier csv
image
A l'étape suivante il me montre les 2 lignes
image
Et quand je clique sur implorter à l'étape suivante, c'est la même page qui se recharge
image

Il n'est pas possible d'aller plus loin. Je suis sur Firefox /W11

@vmarseguerra
Copy link
Contributor Author

Salut @urien

Normalement les problèmes d'édition devraient être résolu.

Pour ce qui est de l'import CSV il semblerait que ce soit un problème de droit d'accès (erreur 403), avec quel utilisateur tu as essayé de faire l'import ? Car il faut qu'il soit administrateur.
Si c'est le cas peux-tu ressayer l'import ? J'ai rajouté plus de log pour mieux comprendre le problème

Copy link
Contributor

@urien urien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oui, c'était bien cela. Merci. Tout est OK pour moi
@bsoufflet est ce que tu veux faire une revue ou est ce que Vincent publie son travail directement ? Je pense que tu vas avoir beaucoup de boulot avec les PR des étudiants durant les 3 prochaines semaines

@vmarseguerra vmarseguerra merged commit 70e1269 into develop Feb 8, 2024
3 of 5 checks passed
@vmarseguerra vmarseguerra deleted the document-refactor branch February 8, 2024 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants