Skip to content

ADS-112 Document Ads channels#744

Draft
MaartenRimaux wants to merge 2 commits into
ADS-110-ads-docs-versioningfrom
ADS-112-document-channels
Draft

ADS-112 Document Ads channels#744
MaartenRimaux wants to merge 2 commits into
ADS-110-ads-docs-versioningfrom
ADS-112-document-channels

Conversation

@MaartenRimaux

Copy link
Copy Markdown
Contributor

Summary

Add the canonical V2 Channels concept and API guide on top of the ADS-110 versioning foundation while leaving frozen V1 content unchanged.

The guide connects channel identity and configuration to origins, breaks, events, templates, marker detection, and integrations; explains wallclock versus PTS scheduling; and documents Dashboard navigation plus equivalent Basic-authenticated CRUD/list/detection examples.

It also records pagination/filter/sort constraints, organization-scoped customAssetKey uniqueness, service-side timing fallbacks, read-only detectionEnabled, and dedicated detection enable/disable endpoints.

Link to Devin session: https://dolby.devinenterprise.com/sessions/b934d3445fec4fd8aa7c8dbdcedfb3bc
Requested by: @MaartenRimaux

devin-ai-integration Bot and others added 2 commits July 16, 2026 13:42
Co-Authored-By: maarten.rimaux <maarten.rimaux@dolby.com>
Co-Authored-By: maarten.rimaux <maarten.rimaux@dolby.com>
@MaartenRimaux MaartenRimaux self-assigned this Jul 16, 2026
@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@github-actions

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://THEOplayer.github.io/documentation/pr-preview/pr-744/

Built to branch gh-pages at 2026-07-16 14:01 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Comment thread sidebarsAds.ts
customProps: {
icon: '📚',
},
items: ['concepts/channels'],

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@MaartenRimaux If you use an auto-generated sidebar, you don't have to stack all of these PRs on top of each other. 😉

Suggested change
items: ['concepts/channels'],
items: [{ type: 'autogenerated', dirName: 'concepts' }],

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good suggestion — an autogenerated concepts sidebar (ordered via sidebar_position frontmatter) would remove the per-PR sidebarsAds.ts edits and naturally keep "Getting started" first. Since this changes the sidebar structure the epic owner approved for the stacked PRs, I've asked @MaartenRimaux to confirm before switching all of them over.

Comment thread sidebarsAds.ts
Comment on lines 17 to 27
{
type: 'category',
label: 'Getting started',
description: 'Set up your first stream with OptiView Ads!',
collapsed: false,
customProps: {
icon: '🚀',
},
link: { type: 'doc', id: 'getting-started/index' },
items: [{ type: 'autogenerated', dirName: 'getting-started' }],
},

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Uhhh shouldn't "Getting started" still be the first category? 🤔

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Agreed — "Getting started" should stay first. I inserted "Core concepts" above it by mistake; I'll move it below "Getting started". (Tracking the exact sidebar structure together with the autogenerated-sidebar suggestion in the sibling thread.)

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