-
Notifications
You must be signed in to change notification settings - Fork 2
(PC-11116): Allow pro user with no structures to reach refunds page #1523
Conversation
31c4fdb
to
66e2e7d
Compare
Codecov Report
@@ Coverage Diff @@
## master #1523 +/- ##
==========================================
+ Coverage 87.87% 87.88% +0.01%
==========================================
Files 377 377
Lines 6400 6406 +6
Branches 1745 1750 +5
==========================================
+ Hits 5624 5630 +6
Misses 748 748
Partials 28 28
Continue to review full report at Codecov.
|
66e2e7d
to
698cc4c
Compare
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.
LGTM : nickel!
une petite proposition pour simplifier la lecture, à prendre ou à laisser
{isLoading ? ( | ||
<Spinner /> | ||
) : ( | ||
venuesOptions.length ? ( | ||
<> | ||
<p> | ||
Les remboursements s’effectuent tous les 15 jours, rétroactivement suite à la validation | ||
d’une contremarque dans le guichet ou à la validation automatique des contremarques | ||
d’évènements. Cette page est automatiquement mise à jour à chaque remboursement. | ||
</p> |
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 pense que c'est pas mal tant que possible d'éviter les if imbriqués, surtout quand l'enfant est aussi long (arrivé en bas de la lecture du composant, on a oublié dans quel test on se trouve)
pour la lecture, peut-être un truc comme :
{isLoading ? ( | |
<Spinner /> | |
) : ( | |
venuesOptions.length ? ( | |
<> | |
<p> | |
Les remboursements s’effectuent tous les 15 jours, rétroactivement suite à la validation | |
d’une contremarque dans le guichet ou à la validation automatique des contremarques | |
d’évènements. Cette page est automatiquement mise à jour à chaque remboursement. | |
</p> | |
{ isLoading && <Spinner /> } | |
{ hasNoResults && (...no result message) } | |
{ hasResults && (le form) } |
et une définition plus haut
const hasNoResults = !isLoading && !venuesOptions.length
const hasResults = !isLoading && !!venuesOptions.length // ou venuesOptions.length > 0
Ajout d'une vérification sur la possibilité d'avoir une liste de venues vide renvoyées par l'API quand l'utilisateur n'a pas de structures rattachées; auquel cas on affiche un message d'informations plutôt qu'avoir un loader infini.