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

Add Plausible. Closes #80 #126

Merged
merged 1 commit into from
Sep 15, 2022
Merged

Add Plausible. Closes #80 #126

merged 1 commit into from
Sep 15, 2022

Conversation

juliangruber
Copy link
Member

Closes #80.

  • Set up Plausible project for app.station.filecoin.io
  • Invite @patrickwoodhead and @bajtos as admins
  • Integrate Plausible script

@bajtos documented these events to be captured, which I think are already covered by default:

Modify Station UI to report the following events

  • a user starts the app (the onboarding flow)

This is a pageview on / or /onboarding or whatever we want to call the route, as long as it isn't the same URL as the dashboard. In the current version, / is onboarding and /saturn is the dashboard.

  • the user closes the app at the screen “Enter FIL”

Plausible lists "exit pages", as long as the "Enter FIL" screen has its own URL this will be taken care of already.

@juliangruber juliangruber requested a review from bajtos September 14, 2022 08:50
Copy link
Member

@bajtos bajtos left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏻

This is a pageview on / or /onboarding or whatever we want to call the route, as long as it isn't the same URL as the dashboard. In the current version, / is onboarding and /saturn is the dashboard.

It would be great to have /onboarding instead of / for onboarding screens. I think that's out of scope of this PR. We can handle this in #75 or later. /cc @jopedroliveira

@juliangruber
Copy link
Member Author

sounds great, we also briefly discussed this in Slack this morning (https://filecoinproject.slack.com/archives/C03N9ENJL77/p1663192253013309)

@juliangruber juliangruber merged commit 07096ee into main Sep 15, 2022
@juliangruber juliangruber deleted the add/plausible branch September 15, 2022 08:35
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.

Usage analytics
2 participants