search

ampproject / amp.dev

The AMP Project Website.
https://amp.dev
Other
582 stars 694 forks source link

Document all component-specific triggers in amp-analytics documentation #2345

Open zhangsu opened 5 years ago

zhangsu commented 5 years ago

Currently, the amp-analytics component reference documentation only mentions component-specific triggers like amp-access-* and amp-video-*.

https://amp.dev/documentation/components/amp-analytics#access-triggers

There are more component-specific triggers like amp-carousel-* for amp-carousel, and amp-form-* for amp-form:

https://github.com/ampproject/amphtml/blob/master/extensions/amp-carousel/amp-carousel-analytics.md

https://github.com/ampproject/amphtml/blob/master/extensions/amp-form/amp-form.md#analytics-triggers

There are amp-story-specific triggers too.

We should add documentation about all the component-specific triggers to the amp-analytics documentation, or at least have a back reference to the component-specific trigger page.

/cc @zhouyx

zhouyx commented 5 years ago

Thanks for filing the issue. Yes I can definitely see the value of documenting all supported triggers at one place. cc @jeffjose

CrystalOnScript commented 4 years ago

@jeffjose @zhouyx Can someone point to where we would be able to find component specific analytics triggers? Is there someone who has a list available?