-
Notifications
You must be signed in to change notification settings - Fork 137
feat: Remove smartlook and add intercom messenger #1175
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request is neither linked to an issue or epic nor labeled as adhoc!
apify-docs-theme/src/theme.js
Outdated
| o.plugins.push(postcssPreset); // allow newest CSS syntax | ||
| return o; | ||
| }, | ||
| injectHtmlTags() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Questing: @webrdaniel can't we use the https://www.npmjs.com/package/@gracefullight/docusaurus-plugin-intercom plugin for this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was kinda discouraged by only 12 weekly downloads but yeah we can use this 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
jbartadev
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm cool with that if it's going to work 👍

The growth team has to config properly the messenger window for docs in Intercom and give us a green light to deploy this.