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

Analytics event: BACK_TO_TOP #2276

Open
fcoveram opened this issue Jun 1, 2023 · 5 comments · May be fixed by #4953
Open

Analytics event: BACK_TO_TOP #2276

fcoveram opened this issue Jun 1, 2023 · 5 comments · May be fixed by #4953
Labels
💻 aspect: code Concerns the software code in the repository 🌟 goal: addition Addition of new feature good first issue New-contributor friendly help wanted Open to participation from the community 🟩 priority: low Low priority and doesn't need to be rushed 🧱 stack: frontend Related to the Nuxt frontend ⌨️ tech: typescript Involves TypeScript 🔧 tech: vue Involves Vue.js

Comments

@fcoveram
Copy link
Contributor

fcoveram commented Jun 1, 2023

Description

The UI component assumes that users need to go back to the top part of Openverse interface, and this change would help us to understand whether that is true or if any other action derivates from this search flow.

Steps:

  1. Add BACK_TO_TOP to the Events type:
    export type Events = {
    • Add query and page payload for the query term and requested page depth respectively. Additionally, scrolledPixels derived from document.getElementById('main-page').scrollTop (or, preferably a Vue ref on that element instead) and similarly maxScroll derived from .scrollTopMax off the same #main-page element.
  2. Fire the event using the useAnalytics composable when the back to top button is clicked. It should be added to https://github.com//WordPress/openverse/blob/HEAD/frontend/src/components/VScrollButton.vue#L47-L50

Additional context

Screenshot of the UI component mentioned

Screenshot of openverse's image results. 'Back to top' is highlighted

Additional information

@fcoveram fcoveram added 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work ✨ goal: improvement Improvement to an existing user-facing feature 🧱 stack: analytics Related to the analytics setup labels Jun 1, 2023
@dhruvkb dhruvkb added 🟩 priority: low Low priority and doesn't need to be rushed 🧹 status: ticket work required Needs more details before it can be worked on 🌟 goal: addition Addition of new feature 💻 aspect: code Concerns the software code in the repository and removed 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work ✨ goal: improvement Improvement to an existing user-facing feature labels Jun 2, 2023
@dhruvkb
Copy link
Member

dhruvkb commented Jun 2, 2023

Added the "🧹 status: ticket work required" label to convert the issue into the same format as the other Analytics event PRs.

@sarayourfriend sarayourfriend changed the title Include the "back to top" action in the usage analytics Analytics event: BACK_TO_TOP Jun 27, 2023
@sarayourfriend sarayourfriend changed the title Analytics event: BACK_TO_TOP Analytics event: BACK_TO_TOP Jun 27, 2023
@sarayourfriend sarayourfriend added good first issue New-contributor friendly 🟨 tech: javascript Involves JavaScript 🔧 tech: vue Involves Vue.js 🧱 stack: frontend Related to the Nuxt frontend help wanted Open to participation from the community and removed 🧹 status: ticket work required Needs more details before it can be worked on 🧱 stack: analytics Related to the analytics setup labels Jun 27, 2023
@nevvada
Copy link
Contributor

nevvada commented Jul 27, 2023

hi @fcoveram, i'd be interested in working on this!

curious, what should we track in this event? i was thinking:

  • search type
  • query/search term
  • result page

@fcoveram
Copy link
Contributor Author

Great, more than welcome. I will refer to @sarayourfriend as I think added the technical info in the main comment.

@sarayourfriend
Copy link
Contributor

Search type does not need to be stored in the payload, it can be inferred from the path (/search is all, /search/image is images, etc).

query and page would make sense, though 👍 I'll update the issue description.

@sarayourfriend
Copy link
Contributor

The description is updated with the payload parameters to add and I've assigned the issue to you @nevvada. Let us know if you need any help 👍

@d3jawu d3jawu linked a pull request Sep 17, 2024 that will close this issue
8 tasks
@obulat obulat added ⌨️ tech: typescript Involves TypeScript and removed 🟨 tech: javascript Involves JavaScript labels Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💻 aspect: code Concerns the software code in the repository 🌟 goal: addition Addition of new feature good first issue New-contributor friendly help wanted Open to participation from the community 🟩 priority: low Low priority and doesn't need to be rushed 🧱 stack: frontend Related to the Nuxt frontend ⌨️ tech: typescript Involves TypeScript 🔧 tech: vue Involves Vue.js
Projects
Status: 🏗 In Progress
Development

Successfully merging a pull request may close this issue.

5 participants