Dev -> Staging v1.1.0 #81
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ensionRuntime protocol
…pdate variables/classes to reflect naming convention changes
Feature merge advertising
…executable Update IdentityConstants with proper values Update comments/documentation Update IdentityStateTests assertion data dictionary access to use IdentityConstants enum values Update copyright year to 2022 in new files Remove dev comments SwiftLint whitespace and indentation correction
…vent and generic Identity event
…d retrieval Add isAdIdEvent check to handleRequestContent Simplify reset identities test case, remove persistedAdID all-zeros test case as there is no migration path to Edge Identity from this state
…in isAdIdEvent check Update documentation for adId
…fication Update adId logic to only provide value sanitization
Implement functional tests for e2e testing
Rename test file Add consent event type and source check, and format verification for testGenericIdentityRequest_whenValidAdId_thenEmptyAdId
Remove logging comments from test file
…sent with old ECID
Moved back shouldDispatchConsent check to original position, pending upstream service discussion
…hould be separate from each other
…nger handled as part of the reset flow
* WIP merge with current dev branch * update enum usage to reflect updated naming scheme * migrate tests from old feature branch, add new files to test targets * Update TestableExtensionRuntime to include new method required by ExtensionRuntime protocol * Migrate remaining test cases from original adID feature branch, and update variables/classes to reflect naming convention changes * Migrate IdentityTest cases * Remove commented code and extra spaces * Update build script for SwiftLint to use PODS_ROOT path to check for executable Update IdentityConstants with proper values Update comments/documentation Update IdentityStateTests assertion data dictionary access to use IdentityConstants enum values Update copyright year to 2022 in new files Remove dev comments SwiftLint whitespace and indentation correction * Update test cases to handle existing AdID and no AdID with new AdID event and generic Identity event * Add isAdIdEvent to Event+Identity extension and build in check to adId retrieval Add isAdIdEvent check to handleRequestContent Simplify reset identities test case, remove persistedAdID all-zeros test case as there is no migration path to Edge Identity from this state * Update Event+Identity extension with rationale for String type check in isAdIdEvent check Update documentation for adId * Update test case to improve readability and preserve event order verification Update adId logic to only provide value sanitization * Remove outdated test for nil adID using direct state.updateAdvertisingIdentifier * Removing public access status for Event+Identity extension property * Update naming conventions for test cases Implement functional tests for e2e testing * Simplify logical flow for detecting ad ID change Rename test file Add consent event type and source check, and format verification for testGenericIdentityRequest_whenValidAdId_thenEmptyAdId * Remove outdated test case with all-zeros start Remove logging comments from test file * Move consent event dispatch to before resetting ECID; should send consent with old ECID * Update adId type in Event extension to non-optional String Moved back shouldDispatchConsent check to original position, pending upstream service discussion * Remove nil coalescing operator for adId after its update to non-optional String * Remove consent event dispatch from resetIdentifiers flow; the flows should be separate from each other * Remove consent event check from resetIdentity test; consent is not longer handled as part of the reset flow * Update test comments and remove commented case
Update comments to better reflect test case goals Update test cases to use variables to store test strings Update naming convention of ad ID value from old -> initial
Update required pods for test app target adding: Edge & Edge Consent Add NSUserTrackingUsageDescription to test app plist
Add get current consent button Update comment and logs to be more accurate
…tion and set the value accordingly
* Update ad ID view to use ad ID utils and better align with sample app implementation * Update requestTrackingAuthorization function to be pure request tracking flow; other functionality is handled by other methods in the util class * Remove get ad ID in request flow; use get ad ID flow instead to fetch or print value * Update documentation wording to defer to apple docs * Update documentation to reflect iOS version verbiage * Updating podfile.lock to use staging * Update logical flow for setting ad ID to check for tracking authorization and set the value accordingly
…iOS 13 compatibility
# Conflicts: # AEPEdgeIdentity.xcodeproj/project.pbxproj
…cene delegates) Apply swiftlint autocorrect and update Button closure syntax to avoid trailing closure violation
Update Swift Test App for iOS 13 compatibility
* Select test scheme from xcodeproj * Select project scheme from xcworkspace * Update test command to use conditional scheme name based on presence of project name conflict or not * Update name conflict case paren string to use EXTENSION_NAME instead of PROJECT_NAME * Update scheme variables to appropriate sources Co-authored-by: Tim Kim <[email protected]>
* Adds documentation for setAdvertisingIdentifier API * Fix typo - identifier
Support for advertising identifier and adId consent collection
Update test app dependencies for adid
TestAppObjC updates for setAdvertisingIdentifier
Bump version to 1.1.0
* [MOB-15816] Implemented getUrlVariables public API and added unit tests for the same (#58) * [MOB-15816] Implemented getUrlVariables public API and unit tests for the same * [MOB-15816] Refactoring and cleanup * [MOB-15816] Refactored url utils * [MOB-15816] Refactored code to simplify util methods for testing. * [MOB-15816] Added getURLVariable API in test app * [MOB-15816] Fixed and added more tests * [MOB-15816] Update codecov patch configuration * [MOB-15816] Review fixes * [MOB-15816] typo fix * [MOB-15816] Minor refactoring and test fixes * [MOB-15816] Update comments and description * [MOB-15816] Minor doc fixes, renaming constants and added getUrlVariables API in objective-c test app * [MOB-15818] Added api-reference in the Documentation directory. Added description and samples for all the APIs along with the new getUrlVariables API. * [MOB-15818] Added extension registration step in the setup section of the documentation. * [MOB-15818] format fixes * [MOB-15818] Review fixes and adding table format for syntax and sample. * [MOB-15818] format fixes * [MOB-15818] Look at different format to represent API sample and syntax * [MOB-15818] Formatting changes * [MOB-15818] More format fixes * trying different formats * format experiments * format changes * check if we can have table without headers in markdown * [MOB-15818] Add objectivec sample for extensionVersion API * [MOB-15818] Add Objective-C samples for getECID API * [MOB-15818] Add dividers after every API for better readability * [MOB-15818] Added ObjectiveC sample for all the APIs and format fixes * [MOB-15818] Format fixes * [MOB-15818] Formated info and warning sections * [MOB-15818] Review fixes * [MOB-15818] Minor fixes * [MOB-15818] Added objective-c samples for public classes (#68) * [MOB-15818] Added objective-c samples for public classes and format fixes * [MOB-15818] Styling and wording fixes * [MOB-15818] Minor format and wording fixes * Merge changes from dev * Remove duplicate enums * Fixed unit test build error from merge with dev * Added functional tests for getUrlVariables API * Review fixes * Using lastSet resolution for configuration shared state to speed up identifier getters * Cleanup in tests to reduce xcode warnings * Fixed an issue with retrieving last set configuration value instead of status * Format fixes for Documentation and bumped up core dependency in podspec (#80) * Format fixes for Documentation and bumped up core dependency in podspec * Documentation fixes and bump up version in Co-authored-by: Emilia Dobrin <[email protected]>
emdobrin
approved these changes
Jun 8, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: