Group pages in react #1220
Group pages in react #1220
Conversation
|
Ok I will not review your code, I'm too old for that but I would like to test it, can you deploy it on staging? |
|
For the "subscribers" option, my idea on this:
|
It is deployed but there is a 404 on the pages. |
rravelli
force-pushed
the
group-page
branch
3 times, most recently
from
936526b to
91c3d24
Compare
| groupTypes?.results.map((groupType) => ({ | ||
| queryFn: () => | ||
| getGroupListApi({ type: groupType.slug, pageSize: PAGE_SIZE }), | ||
| queryKey: ['getGroupList', groupType.slug, isAuthenticated], |
There was a problem hiding this comment.
No need for isAuthenticated in the key
There was a problem hiding this comment.
Ah bon ? Pourtant les groupes dépendent si l'utilisateur est connecté ou non.
There was a problem hiding this comment.
Yes, mais lorsque l'utilisateur se déconnecte le cache de toutes les requêtes est censé être effacé par sécurité (queryClient.clear()) donc c'est inutile de l'ajouter dans la key d'une requêtes (sinon il faudrait ajouter isAuthenticated dans toutes les requêtes vu qu'elles ont toutes une réponse différente si on est connecté ou pas)
There was a problem hiding this comment.
D'accord mais le problème persiste dans la situation inverse où l'utilisateur se connecte, les queries de groupe deviennent alors invalides.
Alors
- soit on clear le cache à la connexion et deconnexion
- soit on le fait uniquement à la deconnexion et on garde les query key isAuthenticated, ce qui est selon moi est une solution bien plus lisible et pas trop coûteuse parce que ça représente pas beaucoup de requêtes. Ca nous permet également de choisir quelle requête refetch lors de la connexion plûtot que de tout supprimer d'un coup...
There was a problem hiding this comment.
Ok je vois, mais je serais plus pour la première solution perso. Je trouve ça plus robuste pour l'avenir : si jamais on oublie d'ajouter la key isAuthenticated sur des requêtes ça pourrait créer des bugs, donc autant s'assurer de tout refetch. Et vu que ya aucune requête (je crois) où on souhaite conserver le cache après la connexion (je vois pas d'exemple où on aurait le même résultat), autant prendre la solution la plus simple
|
Pour avoir une belle grille pour les groupes (je sais plus si c'était fix) : https://youtu.be/3T0gjtXRNC0?si=jfPuEkP0yzJ8G5z4 |
On utilise les |
|
|
Description
The purpose of this PR is to finally implement group pages in React.
The goal is not to implement all the features directly (event though I almost did it), but more to have the pages ready so that we can improve them later.
Changes
Integration of group pages in react.
Changed and added some APIs
New features
Removed
Fixed
Screenshots