Skip to content

Add impression manager and cleanup impressions logic [Part 1]#329

Merged
sanzmauro merged 4 commits intoimpression-dedupe-supportfrom
add-impression-manager-v2
Sep 14, 2020
Merged

Add impression manager and cleanup impressions logic [Part 1]#329
sanzmauro merged 4 commits intoimpression-dedupe-supportfrom
add-impression-manager-v2

Conversation

@sanzmauro
Copy link
Copy Markdown
Contributor

Ruby SDK

What did you accomplish?

  • added impression manager
  • moved impression logic to impr manager

@sanzmauro sanzmauro self-assigned this Sep 10, 2020
@sanzmauro sanzmauro changed the title Add impression manager and cleanup impressions logic Add impression manager and cleanup impressions logic [Part 1] Sep 10, 2020
@sanzmauro sanzmauro changed the base branch from development to impression-dedupe-support September 14, 2020 13:27
@sonarqube-pull-requests
Copy link
Copy Markdown

Kudos, SonarQube Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 9 Code Smells

89.7% 89.7% Coverage
0.0% 0.0% Duplication

@sanzmauro sanzmauro merged commit 53753f0 into impression-dedupe-support Sep 14, 2020
@sanzmauro sanzmauro deleted the add-impression-manager-v2 branch September 14, 2020 14:26
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