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

Remplacer Token.renewal_url par un lien Rails #652

Closed
Samuelfaure opened this issue Aug 17, 2022 · 3 comments · Fixed by #655
Closed

Remplacer Token.renewal_url par un lien Rails #652

Samuelfaure opened this issue Aug 17, 2022 · 3 comments · Fixed by #655
Assignees

Comments

@Samuelfaure
Copy link
Contributor

Cf #651 (comment)

On a un warning brakeman qu'on est obligé d'ignorer à cause de ça

@Samuelfaure Samuelfaure self-assigned this Aug 17, 2022
@Samuelfaure
Copy link
Contributor Author

Problème: Si on veut une route rails il faut à priori obligatoirement utiliser une redirection

il semblerait qu'il y ait un "bug" (ou au moins un design pattern qui empêche le fonctionnement) dans Turbo qui empêche de faire une redirection depuis une turbo-frame: hotwired/turbo#138

Y'a un draft de PR en cours pour gérer ça, et y'a des workaround avec Stimulus, mais ça semble juste pas worth it ici. Je ferme.

@skelz0r
Copy link
Member

skelz0r commented Aug 18, 2022

Problème: Si on veut une route rails il faut à priori obligatoirement utiliser une redirection

Pourquoi ?

Je ne vois pas le rapport avec hotwired/turbo#138 btw

Une autre approche: un helper (qui fait ~office de route). Dtf ça n'a rien à faire dans le modèle.

@Samuelfaure Samuelfaure reopened this Aug 18, 2022
@Samuelfaure
Copy link
Contributor Author

Samuelfaure commented Aug 18, 2022

@skelz0r je suis parti du principe qu'on voulait une route de ce style:

get 'jeton/renew', to: redirect(..datapass)

Je vois pas comment faire différemment pour avoir une route Rails qui redirige vers un autre site?

Mais du coup je vais tenter par helper

Edit: aussi, peut-être l'issue n'est pas en rapport, mais en tout cas j'ai une erreur Typescript quand je tente une redirect depuis la modale turbo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants