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

MoMAGIC Ad Network - Vendor Registration #39898

Closed
wants to merge 23 commits into from
Closed

MoMAGIC Ad Network - Vendor Registration #39898

wants to merge 23 commits into from

Conversation

mmprebid
Copy link
Contributor

✨ New Vendor
MoMAGIC Ad Network is an all-in-one platform that help businesses to generate revenue through Advertisements. MoMAGIC supports advertising technologies like RTB (Real Time Bidding), Header Bidding, Google Ad Manager Based deployment and simplified JS Implementations. We help provide various kinds of Ad Format support ranging from simple banner, video to instream video and interactive banner/video Ads.

This PR adds and integrates the MoMAGIC Ad Network to the amp vendor list, using the tag amp-ad.

@amp-owners-bot amp-owners-bot bot requested a review from powerivq March 14, 2024 08:51
.vscode/settings.json Outdated Show resolved Hide resolved
@erwinmombay
Copy link
Member

@mmprebid could you re-push the code to re-trigger circieCI

extensions/amp-ad/amp-ad.md Outdated Show resolved Hide resolved
.vscode/settings.json Outdated Show resolved Hide resolved
3p/integration.js Outdated Show resolved Hide resolved
@powerivq
Copy link
Contributor

@mmprebid thank you! added two more comments and it should be good then.

@mmprebid mmprebid requested a review from powerivq March 27, 2024 12:44
.vscode/settings.json Outdated Show resolved Hide resolved
extensions/amp-ad/amp-ad.md Outdated Show resolved Hide resolved
.vscode/settings.json Outdated Show resolved Hide resolved
.vscode/settings.json Outdated Show resolved Hide resolved
.vscode/settings.json Outdated Show resolved Hide resolved
@powerivq
Copy link
Contributor

powerivq commented Apr 9, 2024

@mmprebid Thanks. Looks good now, but unfortunately (I have tried a few times), the build is not running. I do not know if it is because of the branch name main or that because I committed something into it. Can you create another branch and open another PR with the same code, and I will approve it there?

@mmprebid
Copy link
Contributor Author

@powerivq
We have created the new PR - #39956 as suggested. Please check and approve accordingly

@erwinmombay
Copy link
Member

closing this PR in favor of the new one

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.

4 participants