Skip to content

Merge dev -> feature/adidConsent#246

Merged
emdobrin merged 2 commits intoadobe:feature/adidConsentfrom
emdobrin:feature/adidConsent
May 9, 2022
Merged

Merge dev -> feature/adidConsent#246
emdobrin merged 2 commits intoadobe:feature/adidConsentfrom
emdobrin:feature/adidConsent

Conversation

@emdobrin
Copy link
Copy Markdown
Contributor

@emdobrin emdobrin commented May 7, 2022

Description

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.

emdobrin and others added 2 commits May 3, 2022 19:10
…adobe#245)

* [DEV] Updates on EdgeEndpoint, renaming, clean up unused fields

[DEV] Renaming EdgeRequestType, environmentType

[DEV] Build edge url with components

[DEV] EdgeEndpoint cleanup

[DEV] Updated tests for impl changes, compacting EdgeEndpointTests

[DEV] Cleanup, docs

[DEV] Fix typo

* [DEV] Remove unused hit getType, fix comment

* Run all tests in foreach

* Fix param name in comment

* [DEV] Rename edgeEndpoint to endpoint

* Renaming to buildEdgeEndpoint
@emdobrin emdobrin requested a review from kevinlind May 7, 2022 00:19
@codecov
Copy link
Copy Markdown

codecov bot commented May 7, 2022

Codecov Report

Merging #246 (8d8bf0e) into feature/adidConsent (a5365f7) will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@                   Coverage Diff                   @@
##           feature/adidConsent     #246      +/-   ##
=======================================================
+ Coverage                94.52%   94.59%   +0.07%     
=======================================================
  Files                       27       27              
  Lines                      985      998      +13     
=======================================================
+ Hits                       931      944      +13     
  Misses                      54       54              

@emdobrin emdobrin requested a review from cacheung May 9, 2022 19:02
@emdobrin emdobrin merged commit b6282d0 into adobe:feature/adidConsent May 9, 2022
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.

2 participants