Event Tracking - Macro Replacement Framework #2623
Closed
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
* OTT-421: Unit tests and basic framework for events consumption and validation * OTT-421: Changed eventvalidation interface according to configuration.validate() requirements * OTT-421: Made validate method from eventvalidation.go private Co-authored-by: Shriprasad <[email protected]>
* OTT-421: Added note to not enabled this feature. Because, it is still in progress Co-authored-by: Shriprasad <[email protected]>
OTT-422: Adding support for eventType=vast in analytic endpoint
…entscoments (#255) * OTT-421: Addressed code review comments Co-authored-by: Shriprasad <[email protected]>
…res) (#256) * OTT-421: Fixed test failures Co-authored-by: Shriprasad <[email protected]>
* OTT-421: Added warning when events.enabled is true Co-authored-by: Shriprasad <[email protected]>
…ents - 2) (#258) * OTT-421: Unit tests and basic framework for events consumption and validation * OTT-421: Reverted unwanted change. * OTT-421: Changed eventvalidation interface according to configuration.validate() requirements * OTT-421: Reverted unwanted change * OTT-421: Made validate method from eventvalidation.go private * OTT-421: Added note to not enabled this feature. Because, it is still in progress * OTT-421: Temporary print statement * OTT-421: Addressed code review comments * OTT-421: Fixed test failures * OTT-421: Added warning when events.enabled is true * OTT-421: Addressed code review comments * OTT-421: Disabled event validations * OTT-421: Reverted stored_requests/data/by_id/accounts/test.json * OTT-421: Removed unwanted comment Co-authored-by: Shriprasad <[email protected]>
* OTT-421: Disabled event validations Co-authored-by: Shriprasad <[email protected]>
…ic-OpenWrap/prebid-server into I1725_vast_event_trackers
* OTT-422: Adding support for eventType=vast in analytic endpoint * Adding test cases
…_event_trackers_staging
* OTT-485: Macro Replacement framework * OTT-485: Changes to support custom macros in req.ext.prebid.macros * Truncating the custom macro value length to 100 chars *Encoding macro values * OTT-485: Adding benchmark * Adding unit tests
SyntaxNode
changed the title
|
Closing this pr because of wrong base branch. Created new pr #2629 |
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.
Issue: #1725
This pr include changes for replacing macros in event tracking urls.