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

[BUG] - Discussions - bouton se connecter #401

Open
narduin opened this issue Oct 14, 2024 · 4 comments · May be fixed by opendatateam/udata-front-kit#606
Open

[BUG] - Discussions - bouton se connecter #401

narduin opened this issue Oct 14, 2024 · 4 comments · May be fixed by opendatateam/udata-front-kit#606
Assignees
Labels
bug Something isn't working

Comments

@narduin
Copy link

narduin commented Oct 14, 2024

Dans les dicsussions, quand on veut démarrer/répondre sans être connecté la fonction suivante délenche une erreur:

const triggerLogin = () => {
  LocalStorageService.setItem('lastRoute', route)
  router.push({ name: 'login' })
}

Peut-être un problème avec le local storage.

--> Le bouton "Connectez-vous pour démarrer une discussion" dans l'onglet discussion d'une page bouquet est inactif.
Comportement attendu : Redirection vers le process d'authentification.
Image

@narduin narduin converted this from a draft issue Oct 14, 2024
@DanFluture275 DanFluture275 added to refine Tickets à présenter en refinement. Retirer le label une fois le ticket passé en backlog ready bug Something isn't working labels Nov 5, 2024
@DanFluture275 DanFluture275 changed the title Discussions - bouton se connecter [BUG] - Discussions - bouton se connecter Nov 5, 2024
@DanFluture275 DanFluture275 removed the to refine Tickets à présenter en refinement. Retirer le label une fois le ticket passé en backlog ready label Nov 6, 2024
@DanFluture275 DanFluture275 moved this to 📋 Backlog Ready in Ecosphères Nov 6, 2024
@DanFluture275 DanFluture275 moved this from 📋 Backlog Ready to To Do in Ecosphères Nov 21, 2024
@at-github
Copy link
Member

Ça va être délicat, je ne reproduis pas en local, sur demo et prod oui par contre

@at-github
Copy link
Member

Le stringify échoue avec le objet route à cause d'une référence circulaire (pas en dev)
Pour l'instant j'ai du mal à mettre le doigt dessus

@at-github at-github linked a pull request Nov 28, 2024 that will close this issue
@at-github at-github moved this from To Do to 👀 In review in Ecosphères Nov 28, 2024
@abulte abulte moved this from 👀 In review to To test in Ecosphères Dec 2, 2024
@DanFluture275
Copy link
Collaborator

DanFluture275 commented Dec 2, 2024

@abulte Pas possible de rediriger directement sur l'onglet Discussions après l'authent, quand on vient du bouton "Connectez-vous pour démarrer une discussion" ?

Sinon, j'ai également vu une erreur de Cors Policy quand on répond à une discussion, il faut cliquer plusieurs fois sur Soumettre pour que ça passe. Hors périmètre de ce ticket, car je reproduis aussi sur d'autres environnements, j'ouvre un nouveau ticket sur ce point.
image

@abulte
Copy link

abulte commented Dec 2, 2024

@abulte Pas possible de rediriger directement sur l'onglet Discussions après l'authent, quand on vient du bouton "Connectez-vous pour démarrer une discussion" ?

Peut-être possible mais pas évident et hors scope du ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: To test
Development

Successfully merging a pull request may close this issue.

4 participants