-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
kulturemedia bid adapter: rebrand to dxkulture #10358
kulturemedia bid adapter: rebrand to dxkulture #10358
Conversation
# Conflicts: # modules/kulturemediaBidAdapter.js # test/spec/modules/kulturemediaBidAdapter_spec.js
kulturemedia-adapter
} from '../src/utils.js'; | ||
import {registerBidder} from '../src/adapters/bidderFactory.js'; | ||
import {BANNER, VIDEO} from '../src/mediaTypes.js'; | ||
|
||
const BIDDER_CODE = 'kulturemedia'; | ||
const BIDDER_CODE = 'dxkulture'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you may want to add your old name as an alias
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the quick review. No publisher are using our old name so it should be fine. Thanks for the heads up
see #10125 for a non-breaking way to rebrand |
@patmmccann I think I explained why we don't need an alias and I hope that's OK. Could you please explain a bit what is a 'non-breaking' way of rebranding? Reading from your reference... does this mean we would have to wait for a new version of Prebid until our rebranding takes effect? Is there something else we can do to change adapter name without breaking the build process? |
Are you saying you have no publishers at all using your adapter? |
@patmmccann Yes, we don't have any active publishers using our adapter at the moment. |
Type of change
Bugfix
Feature
New bidder adapter
Code style update (formatting, local variables)
Refactoring (no functional changes, no api changes)
Build related changes
CI related changes
[X ] Does this change affect user-facing APIs or examples documented on http://prebid.org?
[X ] Other
Description of change
Subject: Rebranding: Changing Adapter Name from kulturemedia to dxkulture
Hi Prebid.js Team,
I hope this message finds you well. I'm reaching out to submit a pull request for the rebranding of our adapter from "kulturemedia" to "dxkulture". This change reflects our new branding and aligns our adapter name more closely with company's identity.
Pull Request Details:
From: kulturemedia
To: dxkulture
Changes Made:
Updated all references to the old adapter name "kulturemedia" to the new name "dxkulture" in the codebase.
Adjusted configuration files to use the new name.
Updated any relevant documentation, comments, and variable names to reflect the rebranding.
Motivation for the Change:
We've recently undergone a rebranding effort, and as part of this process, we're updating our adapter name to "dxkulture". We believe this change will better represent the company and make it easier for users to identify and use our adapter.
Impact Assessment:
This change has been thoroughly tested internally to ensure that it does not introduce any compatibility issues or regressions. All tests have passed successfully, and the adapter continues to work seamlessly with Prebid.js.
Additional Notes:
We have taken care to update any relevant documentation in the repository to reflect the new adapter name. Additionally, the rebranding has been communicated to our existing users to avoid any confusion.
We appreciate your attention to this pull request and your consideration of our rebranding efforts. If there are any further changes or adjustments required, please let us know, and we'll be more than happy to address them promptly.
Thank you for your time and ongoing support.
Best regards,
DXKulture Team