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

Track rss, and add utm_source to each url that generated from feed.js #321

Merged
merged 5 commits into from Sep 16, 2020

Conversation

nonumpa
Copy link
Member

@nonumpa nonumpa commented Aug 26, 2020

Fixes #222

Feature

  1. Track subscriber number by sending event:
    Event Category: RSS
    Event Action: Subscribe
    Event Label: Source(Feedly, Feedrabbit, IFTTT)
    Document title: response header
    Document path: query variables

    Feedly
    • Sends subscriber number when subscribing.
    • Will cache the feed, it triggers the event when first time fetches the feed, and then each regular fetching(maybe once an hour).
    Feedrabbit
    • Sends subscriber number each regular fetching.
    • Triggers the event every time.
  2. Track article page through google tag manager by adding utm_source
    utm_source: Source(Feedly, Feedrabbit, IFTTT)
    utm_medium: RSS

Fix

  1. Copy URL function shouldn't encode URL

Snapshots

Page title: res.header (subscriber number), Page path: query variables

utm_source to article page

@nonumpa nonumpa requested a review from MrOrz September 14, 2020 08:05
@nonumpa nonumpa marked this pull request as ready for review September 14, 2020 08:05
Copy link
Member

@MrOrz MrOrz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the update, it's looking good!

Some thoughts and questions appended.

pages/api/articles/[feed].js Outdated Show resolved Hide resolved
components/Subscribe/FeedDisplay.js Outdated Show resolved Hide resolved
@MrOrz MrOrz merged commit 0e0a3a6 into dev Sep 16, 2020
@MrOrz MrOrz deleted the rss-tracking branch September 21, 2020 06:48
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.

Tracking RSS usage
2 participants