[DEV-12735] Replace current events with new ones using more generic naming #294
Conversation
| export const STORY_IMAGE = { | ||
| CLICK: 'Story Image Click', | ||
| DOWNLOAD: 'Story Image Download', | ||
| VIEW: 'Story Image View', |
There was a problem hiding this comment.
@alpower Previously we've also tracked when image/gallery image was "opened" in stories. Not sure if we need this in Plausible? 🤔
There was a problem hiding this comment.
would it hurt to leave it in? I can see this being nice to know, if not now, at some point.
There was a problem hiding this comment.
I mean.. I guess we could keep it, just seems a bit useless to track that if we're not gonna show it in the dashboard.
There was a problem hiding this comment.
If it's not costing us anything to track it I say keep it, as quite what we show in the dashboard is still up in the air a bit.
There was a problem hiding this comment.
Custom events count as pageviews - we have 10 M events per month and we're already at 0,5 M and it's the 4th day of the month, so in theory it should be okay, but who knows, depends on the amount of images visitors are viewing.
Also, even if we add the Goals to the dashboard later, it won't show past data so I'd say let's skip it for now and I can add it later if we decide that we want this information there, otherwise let's not waste the event limit.
Sounds good?
There was a problem hiding this comment.
yeah sounds good - if it counts towards limits then I agree.
kudlajz
deleted the
feature/dev-12735-implement-plausible-events-in-bea-theme
branch
We want to expose these events to our users in Plausible, so we need better/more generic event names.