-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
OS versions are missing from the CSV export #2624
Comments
Hey @metmarkosaric Currently getting back at Elixir programming, I'm interested in contributing to open-source projects using this language. I'd like to work on this issue if it is still accurate. |
hey @gpouilloux I'm also new to the code, but I looked around a bit and noticed this file: lib/plausible_web/controllers/stats_controller.ex it seems to be responsible for current csv creation. I'm also not sure it was ever creating referrers, since I searched through the history of commits and found nothing. If you still wish to work on it, maybe it will help you somehow. If not, I could take it over. |
another report that the CSV download doesn't include browser versions and OS versions |
Would you like me to work on it? seems like @gpouilloux is busy with something else and not responding here |
Hey @kiote 👋 Good to see you around :) Feel free to work on this, we'd be happy to receive a PR for this. It's fairly well-defined feature addition that doesn't require any architectural changes/decisions. Let me know if you have any questions about it. |
created a PR: #3002 Currently I only was able to test that an empty csv file is created. |
* Add referrers to csv (#2624) * Change CHANGELOG * add missing files * fix tests * revert package files changes * fix typo * format * rename
I'll add Browser / OS support too. Should be pretty much straightforward now |
ok, as I see this ticket is probably got outdated with time :) Browser / OS data is already there: operating_systems.csv
browsers.csv
|
Hi @kiote! It's the browser version numbers and os version numbers that are missing. Basically the numbers that show when you filter the dashboard by a browser or by an os. Thanks! |
hey, will fix that too. Sorry for not being able to read the issue title :D |
Made a change in #3296. Unfortunately looks like all Elixir jobs are failing (locally everything ok, so must be permissions issue). File looks like: browser_version.csv
|
Past Issues Searched
Issue is a Bug Report
Using official Plausible Cloud hosting or self-hosting?
Plausible Cloud from plausible.io
Describe the bug
We've just had a report that our CSV export doesn't include referrers.csv which would include the full list of urls that have sent traffic from a specific source (the list you see in the Top Sources report when you filter the dashboard by a specific source). We currently only have the sources.csv in the export which only has the top level sources. i'm pretty sure that we did use to export the referrers too in the past in its own separate CSV if you click on the export icon after filtering the dashboard by a specific source?
Expected behavior
Export the referrers too in its own separate CSV when you click on the export icon after filtering the dashboard by a specific source
Screenshots
No response
Environment
The text was updated successfully, but these errors were encountered: