-
Notifications
You must be signed in to change notification settings - Fork 334
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
Exclude files from backup #567
Conversation
cc @wcjohnson11 @tonyxiao @ladanazita for review |
and @RetepV as well! |
lgtm! 👍 |
@@ -102,6 +102,9 @@ - (id)initWithAnalytics:(SEGAnalytics *)analytics | |||
dispatch_sync(dispatch_get_main_queue(), ^{ | |||
self.flushTimer = [NSTimer scheduledTimerWithTimeInterval:30.0 target:self selector:@selector(flush) userInfo:nil repeats:YES]; | |||
}); | |||
[self addSkipBackupAttributeToItemAtPath:self.userIDURL]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what about anonymousIdURL
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh good catch, I just copied the ones here https://github.com/segmentio/analytics-ios/blob/46a56f6fe0cd6866f3b6fac370d7815fc341aad0/Analytics/Classes/Internal/SEGSegmentIntegration.m#L498-L500 but look we're missing anonymousID there
* Carthage Support * Initialize flush timer on the main thread. * Replace Foundation import with UIKit import (#563) * Track Campaign Data (#557) * Update contributing docs (#564) * Limit Queue size to 1000 (#565) * Reformat (#566) * Exclude files from backup (#567) * Add tvOS Support (#572) * reformat * Track Deep Links (#573) * Fix CI (#574) * Use vanilla xcodebuild in CI * use xcpretty in ci * Update context object with referrer information (#575) * use 3.2.6 in sample
Closes #537