Add device model for Android analytics

[roozbehzarei]

Device model could be retrieved using the Build class in Android. Refer to this article for more information.

[roozbehzarei]

I see the following information are sent by the Kotlin SDK:
timestamp, sessionId, eventName, systemProps, props. Since systemProps holds the information related to the system, I can send a PR with deviceModel property added to the JSON response in case the website can process the data.

[goenning]

If you want to add that on a PR, I’m happy to merge it. The server will eventually support device model too, but until then it’ll simply ignore it

[roozbehzarei]

I assume the naming of JSON objects and properties are the same across all platforms? If so, what property name do you want me to chose for the device model?

[goenning]

deviceModel seems like a good name to me 👍

[roozbehzarei]

PR: aptabase/aptabase-kotlin#3

Added deviceModel to the systemProps property in the JSON object. Feel free to make contact if you need any additional information added in the future.

[goenning]

I actually wanted to ask if it’s possible to add a Kotlin/Android dependency directly from GitHub, without having to publish to Maven.

and if that’s a common thing to do. The current process to publish to Maven is very tedious

[goenning]

And thanks for the PR, will review soon

[roozbehzarei]

I actually wanted to ask if it’s possible to add a Kotlin/Android dependency directly from GitHub, without having to publish to Maven.

and if that’s a common thing to do. The current process to publish to Maven is very tedious

It's possible and somewhat common. While the most popular libs are all on Maven, there are still many hosted on GitHub. Anyone who wants to import a library from GitHub should add an additional repository in their project and the process doesn't introduce any issues.

If you're interested in doing so, I can make the required changes to the library to make it happen.

[goenning]

Yeah let’s do that! :)

[roozbehzarei]

PR: aptabase/aptabase-kotlin#4

Make sure to read the PR description before merging.

[goenning]

Thanks @roozbehzarei do I need to do anything on https://jitpack.io/ ?

[roozbehzarei]

Sure thing.
No, the process is automatic. Still, I will run a test as soon as the changes are merged to ensure everything works as expected.