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.

Sign up for GitHub

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

Jump to bottom

feat(4243): add doc for create an analystics event from extension #126

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

feat(4243): add doc for create an analystics event from extension #126

wants to merge 1 commit into from
+143 −0

Conversation

DDDDDanica
Copy link
Contributor

Create a new contributor-docs explaining minimum knowledge required to create an analytics event in the MetaMask extension codebase.

@DDDDDanica DDDDDanica self-assigned this Feb 21, 2025
@DDDDDanica DDDDDanica requested a review from a team as a code owner February 21, 2025 02:27
@DDDDDanica DDDDDanica force-pushed the feature/4243-extension branch from d406bb9 to 51f981d Compare February 21, 2025 02:31
@gauthierpetetin
Copy link

Thanks a lot @DDDDDanica for kicking off the tutorial on analytics in Extension!
@NicolasMassart since you have worked a lot on analytics, it would be great to have your insights as well, so I added you as a reviewer. We will also need to include content for analytics in Mobile, probably in the same PR. We can discuss this during the weekly review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NicolasMassart NicolasMassart Awaiting requested review from NicolasMassart

At least 1 approving review is required to merge this pull request.

Assignees

@DDDDDanica DDDDDanica

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DDDDDanica @gauthierpetetin