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

Include the package name in the CSV output #462

Merged
merged 1 commit into from
Oct 12, 2024

Conversation

cvanstone
Copy link
Contributor

I've modified dumpConnectionsCsv and connections_csv_fields to include the package name as well as it's UID and display name. This would be a very helpful addition for our use case.

Repository owner deleted a comment from PmPPolska Oct 10, 2024
@emanuele-f
Copy link
Owner

Can you clarify on your use case?

@cvanstone
Copy link
Contributor Author

I'm hoping to process this data to sit alongside existing application usage stats in a database. This currently accumulates data for multiple devices with a separate table for applications. I've opted to use the package name as the key between this information because:

  • App UID is not common between devices
  • Display name is not unique
  • Package name is unique and consistent between devices

I was looking at post processing the CSV file to add in the package name, but then thought I would see if you would be open to just including the field by default?

@emanuele-f emanuele-f merged commit 2438a33 into emanuele-f:master Oct 12, 2024
4 checks passed
@emanuele-f
Copy link
Owner

Makes sense, thank you!

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