Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

[device_info]add android id #700

Closed
wants to merge 3 commits into from
Closed

[device_info]add android id #700

wants to merge 3 commits into from

Conversation

long1eu
Copy link
Contributor

@long1eu long1eu commented Aug 2, 2018

On iOS we have the identifierForVendor to identify the current device. The Secure.ANDROID_ID is the way to do it in Android.
https://developer.android.com/reference/android/provider/Settings.Secure#ANDROID_ID

@bkonyi bkonyi self-requested a review August 22, 2018 20:55
@@ -119,6 +120,20 @@ class AndroidDeviceInfo {
/// `false` if the application is running in an emulator, `true` otherwise.
final bool isPhysicalDevice;

/// On Android 8.0 (API level 26) and higher versions of the platform,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should have a more terse description for this property on its own line for documentation purposes.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any suggestions?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe something along the lines of this:

"A unique 64-bit encoded identifier provided by the platform"

/// constant for the lifetime of the user's device. On devices that have
/// multiple users, each user appears as a completely separate device,
/// so the ANDROID_ID value is unique to each user.
/// A unique 64-bit encoded identifier provided by the platform.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We still want the more detailed answer, @bkonyi was just saying that we also want a leading paragraph that is the concise description (because our docs software uses the first paragraph in tables of contents and the like).

Can you put back the text you had before after the new paragraph? :-)

@cyanglaz cyanglaz changed the title add android id [deviceinfo]add android id Feb 22, 2019
@cyanglaz cyanglaz changed the title [deviceinfo]add android id [device_info]add android id Feb 22, 2019
@long1eu long1eu closed this May 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants