Skip to content

fix(theme): safely disable announcement bar and refine chat links#157

Merged
hami-robot[bot] merged 1 commit intoProject-HAMi:masterfrom
rootsongjc:channels
Apr 3, 2026
Merged

fix(theme): safely disable announcement bar and refine chat links#157
hami-robot[bot] merged 1 commit intoProject-HAMi:masterfrom
rootsongjc:channels

Conversation

@rootsongjc
Copy link
Copy Markdown
Contributor

Commented out the Docusaurus announcementBar config and added null checks in both the custom AnnouncementBar wrapper and content component to prevent rendering/runtime issues when no announcement is configured.

Also updated community/footer Slack labels to Slack (#hami-dev) (including localized text) and reordered chat links so users know exactly which channel to join.fix(theme): safely disable announcement bar and refine chat links

Signed-off-by: Jimmy Song jimmy@dynamia.ai

Commented out the Docusaurus `announcementBar` config and added null checks in both the custom `AnnouncementBar` wrapper and content component to prevent rendering/runtime issues when no announcement is configured.

Also updated community/footer Slack labels to `Slack (#hami-dev)` (including localized text) and reordered chat links so users know exactly which channel to join.fix(theme): safely disable announcement bar and refine chat links

Signed-off-by: Jimmy Song <jimmy@dynamia.ai>
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 3, 2026

Deploy Preview for project-hami ready!

Name Link
🔨 Latest commit 1734aa4
🔍 Latest deploy log https://app.netlify.com/projects/project-hami/deploys/69cf336ae5a2ef0007aac196
😎 Deploy Preview https://deploy-preview-157--project-hami.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 project configuration.

@hami-robot hami-robot bot added the size/M label Apr 3, 2026
@archlitchi archlitchi self-requested a review April 3, 2026 03:29
Copy link
Copy Markdown
Member

@archlitchi archlitchi left a comment

Choose a reason for hiding this comment

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

/lgtm

@hami-robot
Copy link
Copy Markdown
Contributor

hami-robot bot commented Apr 3, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: archlitchi, rootsongjc

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hami-robot hami-robot bot added the approved label Apr 3, 2026
@hami-robot hami-robot bot merged commit 22bfa92 into Project-HAMi:master Apr 3, 2026
11 checks passed
@rootsongjc rootsongjc deleted the channels branch April 3, 2026 03:42
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.

2 participants