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

crisp(bug): fixing incompatibility with turbo and logoff only when needed #2595

Merged
merged 2 commits into from
Feb 6, 2025

Conversation

Holist
Copy link
Collaborator

@Holist Holist commented Jan 30, 2025

Cette PR règle 2 problèmes :

  • Un problème d'incompatibilité entre Turbo et Crisp (voir l'issue ici)
  • Ne pas appeler la méthode logout quand ce n'est pas nécessaire (quand l'agent n'est pas encore connecté)

@Holist Holist self-assigned this Jan 30, 2025
Copy link
Collaborator

@aminedhobb aminedhobb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci @Holist d'avoir cherché et trouvé un workaround 🙌 !
J'espère que la librairie l'intégrera rapidement pour éviter de garder ce hack..

app/javascript/controllers/crisp_controller.js Outdated Show resolved Hide resolved
app/javascript/controllers/crisp_controller.js Outdated Show resolved Hide resolved
app/views/common/_crisp.html.erb Show resolved Hide resolved
Copy link
Collaborator

@aminedhobb aminedhobb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM merci @Holist pour les changements 👍 , en espérant que le hack ne soit pas là pour trop longtemps!

@Holist Holist merged commit 3db41d3 into staging Feb 6, 2025
6 checks passed
@Holist Holist deleted the fix_issues_with_crisp branch February 6, 2025 09:19
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 this pull request may close these issues.

3 participants