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

Use Matomo for page feedback mechanism #581

Merged
merged 6 commits into from
Apr 26, 2022

Conversation

rossturk
Copy link
Contributor

@rossturk rossturk commented Apr 26, 2022

I was made aware that the page feedback mechanism (i.e., "do you find this page helpful?") in each docs page also uses Google Analytics. As a followup to #576, this moves the mechanism over to Matomo.

Each time a user clicks on the rating button, it will send an event to Matomo. Events in Matomo have a category and an action, along with an optional name and value. I have chosen Docs for the category and Rating as the action - the name is set to the current page path and the value is the numerical rating.

This required a lot of individual file changes in docs-archive. Sorry about the big PR, again! I think it's safe to ignore commit e933c52 :)

Unfortunately, I haven't yet been able to cleanly fix the feedback mechanism for the landing pages. The feature is currently implemented in Docsy, and somewhat tied to GA. For now it has been disabled in this PR.

(This also, as a side effect, more completely deploys the fix in #534. While comparing all of the various docs.js files in the docs archive, I found that they were all the same except for some where that fix had not yet been applied.)

Finally, this bumps the version of the Sphinx theme, so that future doc builds will include the updated mechanism. I'm not sure if the PyPI publication is automatic or not :)

Signed-off-by: Ross Turk <ross@rossturk.com>
Signed-off-by: Ross Turk <ross@rossturk.com>
Signed-off-by: Ross Turk <ross@rossturk.com>
Signed-off-by: Ross Turk <ross@rossturk.com>
Signed-off-by: Ross Turk <ross@rossturk.com>
Signed-off-by: Ross Turk <ross@rossturk.com>
@ashb ashb merged commit 74fb2b7 into apache:main Apr 26, 2022
@potiuk
Copy link
Member

potiuk commented Apr 26, 2022

I just uploaded the PyPI package and restarted the doc build in ~ 20 minutes the latest changes should be live at our page - so worth checking it :)

@rossturk
Copy link
Contributor Author

@potiuk It looks correct to me 👍

@rossturk rossturk deleted the use-matomo-for-feedback branch April 26, 2022 14:52
@potiuk
Copy link
Member

potiuk commented Apr 26, 2022

Cool! thanks!

potiuk pushed a commit that referenced this pull request Jun 17, 2023
* Disable GA-based feedback mechanism in Docsy

* Move to Matomo for page feedback mechanism

* Update feedback mechanism to Matomo in archived docs

* Bump sphinx doc theme

* Switch order of category and action

Signed-off-by: Ross Turk <ross@rossturk.com>
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.

4 participants