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

add config example #5479

Merged
merged 4 commits into from
Jul 14, 2024
Merged

add config example #5479

merged 4 commits into from
Jul 14, 2024

Conversation

DimaIntentIQ
Copy link
Contributor

🏷 Type of documentation

  • new examples

📋 Checklist

Copy link

netlify bot commented Jul 5, 2024

Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit 64cecb0
🔍 Latest deploy log https://app.netlify.com/sites/prebid-docs-preview/deploys/668bf2655badc00008600400
😎 Deploy Preview https://deploy-preview-5479--prebid-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@muuki88 muuki88 left a comment

Choose a reason for hiding this comment

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

Please create a separate analytics module page and link it here.
Analytics modules are documented in https://github.com/prebid/prebid.github.io/tree/master/dev-docs/analytics

dev-docs/analytics/intentiq.md Outdated Show resolved Hide resolved
dev-docs/analytics/intentiq.md Show resolved Hide resolved
Co-authored-by: Muki Seiler <muuki88@users.noreply.github.com>
@muuki88 muuki88 added LGTM and removed needs work labels Jul 8, 2024
@DimaIntentIQ
Copy link
Contributor Author

Hi @muuki88, the PR has been approved. Is there anything else needed from our side before it can be merged?

@muuki88
Copy link
Contributor

muuki88 commented Jul 11, 2024

Hi @DimaIntentIQ

Once released, the docs will be merged 😊

@DimaIntentIQ
Copy link
Contributor Author

Once released, the docs will be merged 😊

Got it, thanks 😊

@DimaIntentIQ
Copy link
Contributor Author

Hi @muuki88, I noticed that the new version with our changes has been released, but this PR hasn't been merged yet. Could you please check it?

@muuki88
Copy link
Contributor

muuki88 commented Jul 14, 2024

Thanks for the ping @DimaIntentIQ

@muuki88 muuki88 merged commit bb7fe85 into prebid:master Jul 14, 2024
5 checks passed
GreDiSe pushed a commit to GreDiSe/prebid.github.io that referenced this pull request Sep 17, 2024
* add config example

* fix: Files should end with a single newline character

* Update dev-docs/analytics/intentiq.md

Co-authored-by: Muki Seiler <muuki88@users.noreply.github.com>

* improve wording

---------

Co-authored-by: Muki Seiler <muuki88@users.noreply.github.com>
Co-authored-by: Eyvaz Ahmadzada <eyvaz.ahmedzade.12@gmail.com>
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.

3 participants