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

Implement Plausible analytics into event statistics #3840

Merged

Conversation

ivarnakken
Copy link
Member

@ivarnakken ivarnakken commented Apr 26, 2023

Description

Implement Plausible analytics into event statistics

Very useful for publicity.

Considering removing "event duration" later since it doesn't really make sense.
Might also implement "bounce rate", but not sure how useful that is.

Result

image

image

Testing

  • I have thoroughly tested my changes.

Since it's a new feature, it should not break anything else.

@ivarnakken ivarnakken added future-is-now Pull requests that utilize cutting-edge technology or just a cool new feature review-needed Pull requests that need review new-feature Pull requests that introduce a new feature labels Apr 26, 2023
@ivarnakken ivarnakken requested a review from a team April 26, 2023 09:24
@ivarnakken ivarnakken self-assigned this Apr 26, 2023
@linear
Copy link

linear bot commented Apr 26, 2023

ABA-428 Use data from Plausible on event statistics

Only the amount of visitors over time. Very useful for PR.

Copy link
Member

@LudvigHz LudvigHz left a comment

Choose a reason for hiding this comment

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

Awesome 💯
Just some small comments. And if it's not planned I still think having the statistics on the entrypoint is probably the most useful here as it will tell us how people actually hear about the different events :)

app/reducers/events.ts Outdated Show resolved Hide resolved
app/routes/events/components/EventAttendeeStatistics.tsx Outdated Show resolved Hide resolved
app/routes/events/components/EventAttendeeStatistics.tsx Outdated Show resolved Hide resolved
@@ -418,3 +419,13 @@ export function isUserFollowing(eventId: number): Thunk<any> {
},
});
}
export function fetchAnalytics(eventId: ID): Thunk<Promise<Action>> {
Copy link
Member

Choose a reason for hiding this comment

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

Since you're using the data from this directly, it would be nice if you could type this properly :)

Copy link
Member Author

Choose a reason for hiding this comment

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

I typed it now, but to do so I had to make a bunch of changes to the call api. Take a look.

Copy link
Member Author

Choose a reason for hiding this comment

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

I removed the typing commit since it didn't work and got out of scope here. You can now find it on branch type-actions.

@LudvigHz
Copy link
Member

I also think you can just remove the visit time thing. It's not very useful at all.

@ivarnakken ivarnakken force-pushed the ivarnakken/aba-428-use-data-from-plausible-on-event branch from 1ed4a9e to 6a62de2 Compare April 27, 2023 16:39
@ivarnakken ivarnakken requested a review from eikhr April 27, 2023 16:44
@ivarnakken ivarnakken force-pushed the ivarnakken/aba-428-use-data-from-plausible-on-event branch 2 times, most recently from 645de7d to 3e37e18 Compare April 27, 2023 17:56
@ivarnakken ivarnakken force-pushed the ivarnakken/aba-428-use-data-from-plausible-on-event branch 2 times, most recently from cdac1be to f163acf Compare April 28, 2023 08:45
app/store/createStore.ts Outdated Show resolved Hide resolved
@ivarnakken ivarnakken force-pushed the ivarnakken/aba-428-use-data-from-plausible-on-event branch from f163acf to 134dfc9 Compare May 2, 2023 13:08
@ivarnakken ivarnakken added approved Pull requests that have been approved and removed review-needed Pull requests that need review labels May 2, 2023
@ivarnakken ivarnakken merged commit 316195b into master May 2, 2023
4 checks passed
@ivarnakken ivarnakken deleted the ivarnakken/aba-428-use-data-from-plausible-on-event branch May 2, 2023 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Pull requests that have been approved future-is-now Pull requests that utilize cutting-edge technology or just a cool new feature new-feature Pull requests that introduce a new feature
Projects
None yet
2 participants