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

✅ [Story analytics] Add test to check sharing happens with the right tag and eventType #37488

Merged
merged 16 commits into from
Jan 27, 2022

Conversation

mszylkowski
Copy link
Contributor

Followup for #37478

@amp-owners-bot
Copy link

amp-owners-bot bot commented Jan 26, 2022

Hey @gmajoulet, @newmuis! These files were changed:

extensions/amp-story/1.0/test/test-amp-story-share.js

@mszylkowski mszylkowski self-assigned this Jan 26, 2022
@mszylkowski mszylkowski merged commit 842f531 into ampproject:main Jan 27, 2022
@mszylkowski mszylkowski deleted the analyticsShareFix branch January 27, 2022 16:21
samouri pushed a commit to samouri/amphtml that referenced this pull request Feb 2, 2022
…tag and eventType (ampproject#37488)

* Added tasts

* Undo

* Adding share examples for analytics

* Using share menu for analytics

* Added test

* Fixed linting

* Fix linting again

* Added comment to retrigger CLA
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants