search

data-players / SyReen

https://app.syreen.fr
0 stars 1 forks source link

pouvoir consulter les données publiques sans être connectés #67

Closed simonLouvet closed 1 year ago

simonLouvet commented 1 year ago

ARbitrage à faire par Syreen

simonLouvet commented 1 year ago

intégrer les discussion sur la visibilité des offres et des profils

VincentFarcy commented 1 year ago

report commentaire @GuillaumeAV depuis issue en doublon https://github.com/data-players/SyReen/issues/80 Quand on est pas connecté, l'objectif serait d'arriver sur l'écran accueil actuel avec la liste des dernières offres publiques + possibilité de rechercher et de naviguer. Il y aurait un bouton "Créer un compte" / "Se connecter" qui fait apparaître le menu du bas, et donne accès à l'ensemble de l'appli et des fonctionnalités. Ca n'a pas l'air de présenter de difficulté ou d'avoir d'impact sur le code selon Seb, juste de l'UX qui peut-être basique :

Arriver non pas sur une écran vide (l'interface d'inscription / connexion) comme à l'heure actuelle mais sur l'écran d'accueil Avoir un bouton "Se connecter" classique sauce React admin => être renvoyé sur l'interface d'inscription / connexion. Quand on clique cliquer l'un ou l'autre des 3 onglets Mes projets / Mon réseau / Mes alertes => être renvoyé sur l'interface d'inscription / connexion. T'évaluerais à combien de temps @VincentFarcy ?

VincentFarcy commented 1 year ago

J'ai fait un 1er essai et lorsqu'on est pas connecté, j'ai un plantage dans findContainersWithTypes le pod est identifié comme un des dataServers même sans être connecté, et l'absence de baseUrl provoque un plantage sur const containerUri = urlJoin(dataServers[key2].baseUrl, path);. il faudrait dans ce cas je suppose ne pas effectuer la recherche de containers. Qu'en penses tu @srosset81 ?

image

srosset81 commented 1 year ago

Oui effectivement. Il y avait du code pour virer le dataserver pod lorsque l'utilisateur n'est pas connecté, mais je l'ai commenté, je sais plus pourquoi:

https://github.com/assemblee-virtuelle/semapps/blob/master/src/frontend/packages/semantic-data-provider/src/dataProvider/utils/fetchUserConfig.js#L43-L46

Ca avait été fait ici:

https://github.com/assemblee-virtuelle/semapps/commit/31f0fda88b6ee39797258365478d5d19011b464e

Il faudrait essayer de décommenter (puis builder) et voir si ça corrige.

VincentFarcy commented 1 year ago

Salut @srosset81, ça fonctionne en réactivant cette partie de code.

VincentFarcy commented 1 year ago

Hello @srosset81, Comment gère t'on la mise à jour de fetchUserConfig.js dans Semapps pour pouvoir merger cette partie dans Syreen ?

srosset81 commented 1 year ago

Tu peux faire une PR. Mais sinon je peux la faire moi lorsque je déploierai l'ensemble...

VincentFarcy commented 1 year ago

@srosset81 ok ça me paraît plus simple si tu t'en occupes.