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

✨(frontend) add crisp chatbot #273

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

arnaud-robin
Copy link
Collaborator

@arnaud-robin arnaud-robin commented Sep 20, 2024

Purpose

We want to add crisp chatbot to allow for easy user interaction with support team.

Proposal

We add the chatbot component using crisp-web-sdk. We initialize the chatbot with the user email to allow for follow-up.

Integrate Crisp chatbot for immediate user support access.

This enables real-time interaction, enhancing user experience
by providing quick assistance.
@arnaud-robin arnaud-robin marked this pull request as ready for review September 20, 2024 16:41
@AntoLC AntoLC force-pushed the feature/crisp-integration branch 3 times, most recently from 2b3e00d to d8b58fe Compare September 23, 2024 13:09
We need to add the Crisp secret at build time
to the frontend container. The frontend is build
from the CI pipeline and the secret is added
to the container as an environment variable.
Copy link
Collaborator

@AntoLC AntoLC left a comment

Choose a reason for hiding this comment

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

I added a few tests, I let you rebase.

We added the key in the secret as well, to be set at build time.

if (!isCrispConfigured()) {
return;
}
Crisp.setTokenId(user.id);
Copy link
Collaborator

@lebaudantoine lebaudantoine Sep 23, 2024

Choose a reason for hiding this comment

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

I am not sure anymore of this part … let's discuss it before merging, I am reading their doc

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

Successfully merging this pull request may close these issues.

3 participants