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 more speaker related charts in analytics #267

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

marcoroth
Copy link
Collaborator

CleanShot 2024-10-22 at 05 30 07@2x

@adrienpoly
Copy link
Owner

Nice!

I am a bit suprise by the shape of the yearly speaking experience. Given that a majority of speakers have 1 talk I would have expected the first column to be higher, but I might be missing something

@marcoroth
Copy link
Collaborator Author

marcoroth commented Oct 22, 2024

I am a bit suprise by the shape of the yearly speaking experience. Given that a majority of speakers have 1 talk I would have expected the first column to be higher, but I might be missing something

There's a good chance that the query is not quite right 😅

But you are right, technically every speakers first talk needs to be in the 0 years column.

@adrienpoly
Copy link
Owner

I propose that we comment this last query and merge it with the others so that we have time to work on it a bit more

or maybe add a poor man feature flag and show it only in dev

@adrienpoly
Copy link
Owner

After some thought I thing we could keep the yearly unique speakers at conferences. The other chart feels less meaningful to me

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.

2 participants