Skip to content

[AMSDK-11328] Check for consent registration based on hub shared state#176

Merged
emdobrin merged 2 commits intoadobe:feature/consentfrom
emdobrin:AMSDK-11328
Mar 25, 2021
Merged

[AMSDK-11328] Check for consent registration based on hub shared state#176
emdobrin merged 2 commits intoadobe:feature/consentfrom
emdobrin:AMSDK-11328

Conversation

@emdobrin
Copy link
Copy Markdown
Contributor

Description

Updated the implementation as this task is now unblocked by adobe/aepsdk-core-ios#570

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 25, 2021

Codecov Report

Merging #176 (87c9dbe) into feature/consent (2ccec67) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@                 Coverage Diff                 @@
##           feature/consent     #176      +/-   ##
===================================================
+ Coverage            93.98%   94.00%   +0.01%     
===================================================
  Files                   25       25              
  Lines                  864      866       +2     
===================================================
+ Hits                   812      814       +2     
  Misses                  52       52              

@emdobrin emdobrin merged commit a523a77 into adobe:feature/consent Mar 25, 2021
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

Successfully merging this pull request may close these issues.

3 participants