Conversation
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>
| textColor: '#fff', | ||
| isCloseable: true, | ||
| }, | ||
| navbar: { |
There was a problem hiding this comment.
🟡 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.
Was this helpful? React with 👍 or 👎 to provide feedback.
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.