Skip to content

feat(community): add ambassadors section to community page - #824

Merged
mesutoezdil merged 5 commits into
Project-HAMi:masterfrom
rootsongjc:feat/ambassador
Sep 3, 2026
Merged

mesutoezdil merged 5 commits into
Project-HAMi:masterfrom
rootsongjc:feat/ambassador

Conversation

@rootsongjc

@rootsongjc rootsongjc commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

What type of PR is this?

/kind feature

What this PR does / why we need it:

Add the current ambassador roster with GitHub links and avatars, a link to the ambassador program doc, and card/grid styles that reuse the maintainers layout.

Which issue(s) this PR fixes:

Checklist:

  • npm run lint and npm run format:check pass
  • npm run build succeeds for both en and zh
  • Chinese translation updated if English docs changed (or noted why not)
  • Commits are signed off (git commit -s)

Summary by CodeRabbit

  • New Features
    • Added an Ambassadors section to the community page featuring five ambassador profiles.
    • Displays ambassador avatars, names, localized locations, GitHub links, and optional LinkedIn and personal website links.
    • Added a link to the ambassador program.
    • Added accessible, icon-based navigation for ambassador profile links.
  • Style
    • Improved responsive layouts for ambassador and maintainer cards, including single-column mobile views.

Add the current ambassador roster with GitHub links and avatars, a link to the ambassador program doc, and card/grid styles that reuse the maintainers layout.

Signed-off-by: Jimmy Song <jimmy@dynamia.ai>
@hami-robot hami-robot Bot added the kind/feature new function label Aug 27, 2026
@hami-robot
hami-robot Bot requested review from wawa0210 and windsonsea August 27, 2026 08:55
@hami-robot

hami-robot Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: rootsongjc
Once this PR has been reviewed and has the lgtm label, please assign archlitchi for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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

@netlify

netlify Bot commented Aug 27, 2026

Copy link
Copy Markdown

Deploy Preview for project-hami ready!

Name Link
🔨 Latest commit 1d2bb69
🔍 Latest deploy log https://app.netlify.com/projects/project-hami/deploys/6a9927ae516aaf0008c4fd26
😎 Deploy Preview https://deploy-preview-824--project-hami.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

Next included review available in 41 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: d3e17675-4c95-430a-bd7a-143b6f58e43c

📥 Commits

Reviewing files that changed from the base of the PR and between d923638 and 1d2bb69.

⛔ Files ignored due to path filters (5)
  • static/img/community/ambassadors/mesut-oezdil.jpg is excluded by !**/*.jpg
  • static/img/community/ambassadors/saiyam-pathak.jpg is excluded by !**/*.jpg
  • static/img/community/ambassadors/shivam-kumar.jpg is excluded by !**/*.jpg
  • static/img/community/ambassadors/tianqing-wang.jpg is excluded by !**/*.jpg
  • static/img/community/ambassadors/xueduan-li.jpg is excluded by !**/*.jpg
📒 Files selected for processing (3)
  • src/data/ambassadors.json
  • src/pages/community.js
  • src/pages/community.module.css

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 0dbba0c7-c19b-453e-beec-f742afcad2c1

📥 Commits

Reviewing files that changed from the base of the PR and between 9f2b6b8 and d923638.

📒 Files selected for processing (3)
  • src/data/ambassadors.json
  • src/pages/community.js
  • src/pages/community.module.css

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The community page loads five ambassador profiles from JSON. It renders localized locations and optional LinkedIn and Website links. Shared CSS styles maintainer and ambassador cards, including responsive single-column layouts.

Changes

Community ambassador program

Layer / File(s) Summary
Ambassador data and rendering
src/data/ambassadors.json, src/pages/community.js
The page loads five ambassador records, resolves avatar paths with useBaseUrlUtils, and renders localized profile cards with optional LinkedIn and Website links.
Shared ambassador card styling
src/pages/community.module.css
Shared selectors style program links, maintainer and ambassador cards, hover states, card content, avatars, metadata, icon links, and mobile grids.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: ⚪ Minimal · up to d9236

The community page adds localized ambassador cards with profile links and responsive styling. No concrete current-head issue remains that would block merge.

Sequence Diagram(s)

sequenceDiagram
  participant CommunityPage
  participant AmbassadorData
  participant BaseUrlUtils
  participant AmbassadorCard
  CommunityPage->>AmbassadorData: Load ambassador profiles
  CommunityPage->>BaseUrlUtils: Resolve avatar paths
  CommunityPage->>AmbassadorCard: Render localized profile data and links
Loading
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 1 files. (2 skipped: 2 … Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: adding an ambassadors section to the community page.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 1 files. (2 skipped: 2 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@mesutoezdil mesutoezdil left a comment

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.

thanks for the pr

@mesutoezdil mesutoezdil left a comment

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.

few inline notes.

Comment thread src/pages/community.js Outdated
Comment thread src/pages/community.js Outdated
Comment thread src/pages/community.js Outdated
Comment thread src/pages/community.js
Comment thread src/pages/community.js
Comment thread src/pages/community.js Outdated
…e links

Signed-off-by: Jimmy Song <jimmy@dynamia.ai>
Comment thread src/pages/community.js Outdated
Comment thread src/data/ambassadors.json Outdated
Comment thread src/data/ambassadors.json Outdated
Comment thread src/pages/community.module.css Outdated
…LinkedIn link; enhance ambassador layout and styles

Signed-off-by: Jimmy Song <jimmy@dynamia.ai>
Comment thread src/pages/community.js Outdated
Comment thread src/pages/community.js
Signed-off-by: Jimmy Song <jimmy@dynamia.ai>
Signed-off-by: Jimmy Song <jimmy@dynamia.ai>
@mesutoezdil

Copy link
Copy Markdown
Contributor

/lgtm

all three points are covered.

the nav landmarks are gone, the card styles are shared through .personCard and friends so the two lists cannot drift apart any more, and the avatars are regenerated at 160x160, about 6 KB each. that is the same size the maintainer cards request from github, and total weight drops from roughly 178 KB to 32 KB.

keeping the files committed rather than pointing at the github avatar is a fair call, it means the project controls which image shows.

@hami-robot hami-robot Bot added the lgtm label Sep 3, 2026
@mesutoezdil
mesutoezdil merged commit b24ffd7 into Project-HAMi:master Sep 3, 2026
9 of 10 checks passed
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