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

Allow DeviceData to be set #95

Open
ptsteele opened this issue Nov 18, 2022 · 1 comment
Open

Allow DeviceData to be set #95

ptsteele opened this issue Nov 18, 2022 · 1 comment

Comments

@ptsteele
Copy link

ptsteele commented Nov 18, 2022

For our application we're looking to manually control the device information that is recorded for a session. It appears that all the fields we'd like to change exist but are locked behind internal classes.

https://github.com/Countly/countly-sdk-windows/blob/master/countlyCommon/countlyCommon/CountlyBase.cs#L141

Is there any chance there could be an option to set the values in Device.cs through a configuration, or at a minimum remove the internal tag from these fields / classes so they could be extended and overwritten by consuming applications?

I'd be happy to whip up a PR if that is an acceptable idea but there is not bandwidth for it.

@ZahidZafar
Copy link
Contributor

Hello @ptsteele ,

It sounds like you are referencing the fields that are sent under the "metric" parameter when sending together a request that sends the "begin_session" parameter. The ability to override these values is something that we want to add, and it currently already was on our roadmap.

Thank you for offering to create a PR for this, but we'll be implementing and releasing this ourselves in the next 1-2 months.

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

No branches or pull requests

3 participants