-
Notifications
You must be signed in to change notification settings - Fork 32
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
Docs: Add Ratings functionality #1038
Conversation
38cc492
to
cd9f0db
Compare
Hello! 👋 This repository uses Auto for releasing packages using PR labels. ✨ This PR can be merged. It will not be considered when calculating future versions of the npm packages and will not appear in the changelogs. |
@@ -74,6 +74,12 @@ const config = { | |||
sdkUrl: 'https://rsdk.grafana.com', | |||
}, | |||
canSpamUrl: 'https://grafana-dev.com/canspam', | |||
faroConfig: { | |||
url: '/connect/af1fca71911a9641ebdffddb56889e97', |
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 think this is a production key, I don't think there is one for dev
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.
Generally great addition, I am not 100% sure we will keep using the same way as website does but for now it's fine to attach ourselves to them. I am mostly thinking about data retention which is way higher if we just use rudderstack.
What this PR does / why we need it:
This PR introduces a Ratings component to allow users to give feedback on how useful they find a page on our docs site.
The main docs website uses faro which I've brought across for the ride. I'm not sure if the behaviour is correct that faro is only setup and sends measurements if the cookies are agreed to (same as rudderstack).
What does it look like:
Original buttons state:
Once rated buttons state:
Which issue(s) this PR fixes:
Fixes #1006
Special notes for your reviewer: