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

the events count in a session detail page is not right #2980

Closed
pinkli opened this issue Sep 24, 2024 · 2 comments
Closed

the events count in a session detail page is not right #2980

pinkli opened this issue Sep 24, 2024 · 2 comments
Labels
bug Something isn't working fixed in dev Fixed in the dev branch

Comments

@pinkli
Copy link

pinkli commented Sep 24, 2024

Describe the Bug

the count is the same as views, see code

sum(case when website_event.event_type = 1 then 1 else 0 end) as events

should change to case when website_event.event_type = 1 then 0 else 1 end

Database

PostgreSQL

Relevant log output

No response

Which Umami version are you using? (if relevant)

master branch

Which browser are you using? (if relevant)

No response

How are you deploying your application? (if relevant)

No response

@franciscao633 franciscao633 added the bug Something isn't working label Sep 24, 2024
@franciscao633
Copy link
Collaborator

Feel free to put in a PR, but it should be the below. We currently only have two event types, but targeting event_type = 2, should protect us if we add more. Thanks for catching this.

sum(case when website_event.event_type = 2 then 1 else 0 end) as events

mikecao added a commit that referenced this issue Sep 26, 2024
bugfix: events count amend #2980
@franciscao633 franciscao633 added the fixed in dev Fixed in the dev branch label Sep 30, 2024
@mikecao
Copy link
Collaborator

mikecao commented Nov 22, 2024

Fixed in v2.14.0.

@mikecao mikecao closed this as completed Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed in dev Fixed in the dev branch
Projects
None yet
Development

No branches or pull requests

3 participants