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

[TIMOB-10787] iOS: Expose modern UUID-replacements #8953

Merged
merged 11 commits into from
Nov 13, 2017

Conversation

hansemannn
Copy link
Collaborator

#if defined(USE_TI_PLATFORMIDENTIFIERFORADVERTISING) || defined(USE_TI_PLATFORMGETIDENTIFIERFORADVERTISING)
- (NSString *)identifierForAdvertising
{
return [[[ASIdentifierManager sharedManager] advertisingIdentifier] UUIDString];
Copy link
Contributor

Choose a reason for hiding this comment

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

Apple here suggest, we should use this by checking isAdvertisingTrackingEnabled. We can expose "isAdvertisingTrackingEnabled" property as well . Ygor Lemos has also suggested the same in comment.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good idea, added it!

}

#if defined(USE_TI_PLATFORMIDENTIFIERFORADVERTISING) || defined(USE_TI_PLATFORMGETIDENTIFIERFORADVERTISING)
- (NSNumber *)isAdvertisingTrackingEnabled
Copy link
Contributor

Choose a reason for hiding this comment

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

Make "isAdvertisingTrackingEnabled" property in .h file similar to
+@Property(readonly,nonatomic) NSString *identifierForAdvertising;

@hansemannn hansemannn modified the milestones: 7.0.0, 6.2.0 Aug 15, 2017
@vijaysingh-axway
Copy link
Contributor

@hansemannn Can you resolve conflicts please.

… TIMOB-10787

# Conflicts:
#	apidoc/Titanium/Platform/Platform.yml
#	iphone/Classes/PlatformModule.h
#	iphone/Classes/PlatformModule.m
#	iphone/Classes/defines.h
@hansemannn
Copy link
Collaborator Author

hansemannn commented Nov 10, 2017

@vijaysingh-axway Restored and added unit-tests

Copy link
Contributor

@vijaysingh-axway vijaysingh-axway left a comment

Choose a reason for hiding this comment

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

CR and FT passed.

@ewieberappc
Copy link
Contributor

FR Passed. IdentifierForVendor and IdentifierForAdvertising return the appropriate values. Tested on multiple sims/devices with different apps and with advertising tracking enabled and disabled.

@eric34 eric34 merged commit 17fa7e2 into tidev:master Nov 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants