Skip to content

Use vidazoo utils#11879

Closed
uditalias wants to merge 2 commits intoprebid:masterfrom
Vidazoo:use_vidazoo_utils
Closed

Use vidazoo utils#11879
uditalias wants to merge 2 commits intoprebid:masterfrom
Vidazoo:use_vidazoo_utils

Conversation

@uditalias
Copy link
Contributor

Type of change

  • Refactoring (no functional changes, no api changes)

Description of change

Refactoring ShinezRTB, Tagoras, TwistDigital, KueezRTB bid adapters to use the shared vidazooUtils in order to remove code duplications.

Other information

#11865

@github-actions
Copy link

Whoa there, partner! 🌵🤠 We wrangled some duplicated code in your PR:

Reducing code duplication by importing common functions from a library not only makes our code cleaner but also easier to maintain. Please move the common code from both files into a library and import it in each. Keep up the great work! 🚀

break;
case 'digitrustid':
payloadRef[key] = deepAccess(userId, 'data.id');
break;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These don't exist any longer

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

Comments