Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tnc Id System : fixes for docs and performance improvements #12315

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

annavane
Copy link
Contributor

Pull Request contains following bug fixes and changes:

modules/tncIdSystem.js
Optimized User ID Recovery: Replaced the existing user ID recovery function with a faster and more efficient method, improving performance.

modules/userId/userId.md
Documentation Correction: Resolved inconsistencies in the documentation, ensuring accurate information for module configuration and usage.


Type of change

  • [x ] Bugfix

  • Feature

  • New bidder adapter

  • Updated bidder adapter

  • Code style update (formatting, local variables)

  • Refactoring (no functional changes, no api changes)

  • Build related changes

  • CI related changes

  • Does this change affect user-facing APIs or examples documented on http://prebid.org?

  • Other

Description of change

Other information

modules/tncIdSystem.js  - Optimized User ID Recovery: Replaced the existing user ID recovery function with a faster and more efficient method, improving performance.
modules/userId/userId.md - Documentation Correction: Resolved inconsistencies in the documentation, ensuring accurate information for module configuration and usage.
@ChrisHuie ChrisHuie changed the title tncIdSystem - Bug Fixes tnc Id System : fixes Oct 10, 2024
@ChrisHuie ChrisHuie changed the title tnc Id System : fixes tnc Id System : fixes for docs and performance improvements Oct 10, 2024
@ChrisHuie
Copy link
Collaborator

@annavane looks like you have some failing unit tests

@annavane
Copy link
Contributor Author

Hi, tests are ok now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants