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

feat: autocomplete dataLayer events #203

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

joaocore
Copy link

@joaocore joaocore commented Nov 13, 2024

What is the purpose of this pull request?

Enviar eventos de dentro do autocomplete para o dataLayer.

What problem is this solving?

O código atual já tinha uma estrutura inicial para enviar os eventos dentro do autocomplete, porém, esses eventos não estavam sendo enviados, como:

  • Clique no termo de sugestões
  • Clique no termo de histórico de busca
  • Clique no produto
  • etc

Com os ajustes, os seguintes eventos agora são disparados:

top_search_click: quando um dos termos mais buscados é clicado.
history_click: quando um dos termos do histórico de busca é clicado.
search_suggestion_click: quando é clicado em algum termo de sugestão.
see_all_products_click: quando é clicado na opção ver todos os produtos, se houver product-summary.
productClick: quando é clicado sobre algum produto.

handleAutocompleteSearch: descontinuei, pois não estava enviando eventos search e acabava sobrepondo qualquer um dos eventos acima, enviando os dados vazios.

How should this be manually tested?

top_search_click: clique no input da barra de busca e em seguida clique em um dos termos mais buscados.
history_click: clique no input da barra de busca e em seguida clique em dos termos que você já pesquisou e que está no histórico de busca,
search_suggestion_click: clique no input da barra de busca e em seguida digite algum termo. Algumas sugestões de termos irão aparecer, clique em alguma delas.
see_all_products_click: clique no input da barra de busca e em seguida digite algum termo. Na shelf, clique em ver todos os produtos (caso tenha).
productClick: clique no input da barra de busca e em seguida digite algum termo. Na shelf, clique sobre algum produto (caso tenha).

Essa PR também tem dependências de outras PRs:
vtex-apps/google-tag-manager#153
vtex-apps/pixel-manager#39

Types of changes

  • Bug fix (a non-breaking change which fixes an issue)
  • New feature (a non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Requires change to documentation, which has been updated accordingly.

Copy link

vtex-io-ci-cd bot commented Nov 13, 2024

Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖

Please select which version do you want to release:

  • Patch (backwards-compatible bug fixes)

  • Minor (backwards-compatible functionality)

  • Major (incompatible API changes)

And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.

  • No thanks, I would rather do it manually 😞

@vtex-io-docs-bot
Copy link

Beep boop 🤖

I noticed you didn't make any changes at the docs/ folder

  • There's nothing new to document 🤔
  • I'll do it later 😞

In order to keep track, I'll create an issue if you decide now is not a good time

  • I just updated 🎉🎉

@joaocore
Copy link
Author

Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖

Please select which version do you want to release:

  • Patch (backwards-compatible bug fixes)
  • Minor (backwards-compatible functionality)
  • Major (incompatible API changes)

And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.

  • No thanks, I would rather do it manually 😞

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.

1 participant