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

Log user's screen resolution #2507

Merged
merged 1 commit into from
May 2, 2018
Merged

Log user's screen resolution #2507

merged 1 commit into from
May 2, 2018

Conversation

arikfr
Copy link
Member

@arikfr arikfr commented May 2, 2018

No description provided.

@arikfr arikfr merged commit bce0e8e into master May 2, 2018
@arikfr arikfr deleted the resolution branch May 2, 2018 11:09
@kyoshidajp
Copy link
Member

@arikfr I think this screen_resolution will be logged to an events table, but the table doesn't have the field. Could you tell me where can I see this data?

@arikfr
Copy link
Member Author

arikfr commented May 13, 2018

It will be in the additional_properties column, here's an example query:

select additional_properties::json->>'screen_resolution', count(distinct user_id) as unique_users, count(0) as events_count
from events
where created_at > current_date - 7 and additional_properties::json->>'screen_resolution' <> ''
group by 1

@kyoshidajp
Copy link
Member

@arikfr Thanks. I agree to that.

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