Skip to content

Rebranding and language updates#591

Open
dcoffey3296 wants to merge 15 commits intomainfrom
language-updates
Open

Rebranding and language updates#591
dcoffey3296 wants to merge 15 commits intomainfrom
language-updates

Conversation

@dcoffey3296
Copy link
Copy Markdown
Collaborator

@dcoffey3296 dcoffey3296 commented Apr 10, 2026

Rebranding updates heading into NAB. Removed the announcement bar, added branding callouts, unified millicast/theoplayer/theolive/theoads/dolby.io -> OptiView. Code and package names remain the original names.


Open with Devin

devin-ai-integration[bot]

This comment was marked as resolved.

dcoffey3296 and others added 2 commits April 10, 2026 16:19
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 new potential issue.

View 8 additional findings in Devin Review.

Open in Devin Review

textColor: '#fff',
isCloseable: true,
},
navbar: {
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.

🟡 PR preview announcement bar inadvertently removed along with rebranding announcement

The entire announcementBar config was removed from docusaurus.config.ts, but this contained two separate concerns bundled in a ternary expression: (1) a rebranding notice for production, and (2) a PR preview warning for PR preview deployments. The CI workflow at .github/workflows/pr-preview.yml:47-49 still sets DOCUSAURUS_PR_NUMBER and DOCUSAURUS_PR_URL environment variables expecting this announcement bar to render, but the code consuming these variables is now gone. As a result, PR preview deployments will no longer display the warning banner ("This is a preview of the documentation website from pull request #...") that alerts users the content may be inaccurate.

Prompt for agents
The announcementBar config in docusaurus.config.ts themeConfig was entirely removed. While removing the rebranding announcement ('dolby_optiview_new_name') was intentional, the PR preview announcement bar was also removed as collateral damage because both were in a single ternary expression.

The PR preview workflow (.github/workflows/pr-preview.yml lines 47-49) still sets DOCUSAURUS_PR_NUMBER and DOCUSAURUS_PR_URL environment variables and has a comment saying 'Add an announcement at the top to indicate that this is a preview'.

To fix this, re-add only the PR preview announcement bar to the themeConfig in docusaurus.config.ts. The property should be added right before the 'navbar' property. Something like:

announcementBar: process.env.DOCUSAURUS_PR_NUMBER
  ? {
      id: 'pr_preview',
      content: `This is a preview of the documentation website from <a target="_blank" rel="noopener noreferrer" href="${process.env.DOCUSAURUS_PR_URL}">pull request #${process.env.DOCUSAURUS_PR_NUMBER}</a>. It may contain inaccurate or incomplete information. Do not rely on it for official documentation.`,
      backgroundColor: '#9cb9c9',
      textColor: '#344a5e',
      isCloseable: false,
    }
  : undefined,

This preserves the PR preview warning while still removing the old rebranding announcement bar.
Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

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.

1 participant