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

Test RC Events #637

Closed
wants to merge 1 commit into from
Closed

Test RC Events #637

wants to merge 1 commit into from

Conversation

lariciamota
Copy link
Contributor

What's the purpose of this pull request?

How does it work?

How to test it?

Faststore related PRs

References

Copy link

vercel bot commented Dec 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
starter ✅ Ready (Inspect) Visit Preview Dec 13, 2024 6:35pm

lariciamota added a commit to vtex/faststore that referenced this pull request Dec 19, 2024
## What's the purpose of this pull request?

Add VTEX RC Events to FastStore.
- HomeView
- CategoryView
- DepartmentView
- InternalSiteSearchView
- ProductView
- OtherView

## How it works?

The events weren't being sent to RC using the arguments expected by RC,
so now the new `handleEvent` function does that - depending on the type
of the page, it'll send the correct page view event to RC.

## How to test it?

Run `yarn dev` on this branch and watch the Network tab for the `v8`
request. I'll add some screenshots below to demonstrate:

| Event | Screenshot |
| --- | --- |
| homeView | <img width="1438" alt="homeView"
src="https://github.com/user-attachments/assets/36006b36-226f-4742-b166-54b8120d2d07"
/> |
| productView | <img width="1435" alt="productView"
src="https://github.com/user-attachments/assets/27be74bc-2595-4da1-b72c-f6e5254def10"
/> |
| categoryView | <img width="1437" alt="categoryView"
src="https://github.com/user-attachments/assets/a6e415c5-05af-4fe8-b3d3-7b4961121a73"
/> |
| departmentView | <img width="1438" alt="departmentView"
src="https://github.com/user-attachments/assets/a560f4fc-48e7-43aa-aaf1-ba400cb002c1"
/> |
| internalSiteSearchView | <img width="1438"
alt="internalSiteSearchView"
src="https://github.com/user-attachments/assets/3086274a-559a-434f-93c8-1baaa85c5284"
/> |
| otherView | <img width="1434" alt="otherView"
src="https://github.com/user-attachments/assets/b70c91cf-7339-463a-a19a-80507b9c22ce"
/> |

### Starters Deploy Preview

https://starter-5wjtxcga4-vtex.vercel.app/ /
https://sfj-f304385--starter.preview.vtex.app/
([PR](vtex-sites/starter.store#637))

## References

- [Jira task](https://vtex-dev.atlassian.net/browse/SFS-1796)
- [Slack
thread](https://vtex.slack.com/archives/C051B6LL91U/p1730123029648129)
- [RC app](https://github.com/vtex/request-capture-app)
- [RC script](https://github.com/vtex/request-capture-script)
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