[Export] Créer une page d'export et sortir les liens des dropdowns #9473
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lisa-durand
force-pushed
the
create-export-page-V2-ldu
branch
from
168c7cd to
7b3092f
Compare
colinux
force-pushed
the
create-export-page-V2-ldu
branch
2 times, most recently
from
2618f64 to
5603515
Compare
lisa-durand
force-pushed
the
create-export-page-V2-ldu
branch
2 times, most recently
from
4b9aa9e to
56ec373
Compare
colinux
force-pushed
the
create-export-page-V2-ldu
branch
4 times, most recently
from
1478d16 to
4c26fae
Compare
mfo
reviewed
Sep 21, 2023
mfo
reviewed
Sep 21, 2023
mfo
reviewed
Sep 21, 2023
app/components/dossiers/export_link_component/export_link_component.html.haml
Outdated
Show resolved
Hide resolved
mfo
requested changes
Sep 21, 2023
There was a problem hiding this comment.
déjà bravo car l'ui/l'ux me semble canon 😍
Aussi, en sorti d'UI/UX canon, c'est vraiment top cette page qui liste TOUS les exports car ça evite la gymnastique/compléxité bizarre du find_for_groupe_instructeurs et ce qui découlait derrière avec les boutons etc... 👏
Malheureusement en test je trouve quelques point bloquants :
- le point de secu (cf le patch via mattermost pour pas polluer votre taff)
- le fait de pouvoir regenerer un export quand il est fail (deux commentaires la dessus).
- quand je demande la création d'un export, deux exports s'affichent dans mon UI
- a clarifier la difference entre le bouton pour télécharger tous les dossiers (incluant ceux archivés, donc bouton en haut de page) et le bouton pour télécharger tous les dossiers de l'onglet au total (qui n'inclut pas les dossiers archivés)
|
C'est magnifique ! "Export .ods d'e 5 dossiers au total" , je ne comprends pas si c'est un export de tous les dossiers au total, si c'est 5 dossiers parmi d'autre. et du coup, moi qui suis un instructeur qui veut tous les dossiers, je ne sais pas si je peux réutiliser ce lien ou si je dois faire un nouvel export. |
|
J'ai corrigé les problèmes remontés par @mfo et amélioré le wording général suite à vos retours (cf screenshot qui montre plusieurs cas possibles) :
Il reste la discussion plus haut sur la question de regénérer un export |
colinux
force-pushed
the
create-export-page-V2-ldu
branch
from
5077758 to
2cdd0cc
Compare
…rts components Co-Authored-By: Lisa Durand <lisa.c.durand@gmail.com>
…ts dropdown Since the dropdown does not list download links anymore, we don't have to pass them a dynamic exports list. Co-Authored-By: Lisa Durand <lisa.c.durand@gmail.com>
… export Il semblerait que le destroy échoue parfois silencieusement pour une raison inconnue, ce qui empêche le create_or_find_export de se dérouler à cause de la contrainte d'unicité (il rollback), ce qui conduit ensuite à un comportement inattendu: re-téléchargement de l'ancienne archive alors qu'on voulait la regénérer. Co-Authored-By: Lisa Durand <lisa.c.durand@gmail.com>
Co-Authored-By: Lisa Durand <lisa.c.durand@gmail.com>
Co-Authored-By: Lisa Durand <lisa.c.durand@gmail.com>
…erated Co-Authored-By: Lisa Durand <lisa.c.durand@gmail.com>
colinux
force-pushed
the
create-export-page-V2-ldu
branch
2 times, most recently
from
a3b7d28 to
8b26dcc
Compare
Co-Authored-By: Lisa Durand <lisa.c.durand@gmail.com>
…te a fresh one Pour permettre d'avoir des données fraiches dans un export tout en conservant l'historique des exports, la demande d'export depuis le menu créé toujours un nouvel export sauf: - si un autre export identique est déjà en préparation - si un autre export identique s'est terminé il y a moins de 5 minutes Co-Authored-By: Lisa Durand <lisa.c.durand@gmail.com>
colinux
force-pushed
the
create-export-page-V2-ldu
branch
from
8b26dcc to
d419886
Compare
tchak
approved these changes
Sep 28, 2023
mfo
approved these changes
Sep 28, 2023
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
closes #9368
Reste à faire