Skip to content

Add EcosystemList component and integrate new ecosystem data#514

Open
rootsongjc wants to merge 3 commits into
Project-HAMi:masterfrom
rootsongjc:feat/integrations
Open

Add EcosystemList component and integrate new ecosystem data#514
rootsongjc wants to merge 3 commits into
Project-HAMi:masterfrom
rootsongjc:feat/integrations

Conversation

@rootsongjc

@rootsongjc rootsongjc commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

Introduce the EcosystemList component to display various integrations and add Qwen to the ecosystem. Update styles to accommodate the new layout and ensure proper rendering of logos. Reintroduce the ray.svg logo with its original content.

Summary by CodeRabbit

  • New Features

    • Added a new “Works with HAMi” ecosystem section on the home page.
    • Introduced an ecosystem list with localized names and logos, plus links to project websites when available.
    • Expanded the ecosystem showcase with additional entries, including Qwen and KAI Scheduler.
  • Layout

    • Updated the ecosystem grid to display more items per row across screen sizes for a denser, more balanced layout.

…showcase

Signed-off-by: Jimmy Song <jimmy@dynamia.ai>
- Added Qwen with its logo to the hero workload ecosystem in index.js.
- Updated styles.module.css to accommodate an additional column in the ecosystem grid.
- Added new SVG file for Qwen logo.
- Reintroduced ray.svg with its original content.

Signed-off-by: Jimmy Song <jimmy@dynamia.ai>
Signed-off-by: Jimmy Song <jimmy@dynamia.ai>
@hami-robot

hami-robot Bot commented Jun 28, 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

@hami-robot hami-robot Bot requested review from archlitchi and windsonsea June 28, 2026 11:34
@netlify

netlify Bot commented Jun 28, 2026

Copy link
Copy Markdown

Deploy Preview for project-hami ready!

Name Link
🔨 Latest commit 6840b29
🔍 Latest deploy log https://app.netlify.com/projects/project-hami/deploys/6a4106a88d9f2700087777c3
😎 Deploy Preview https://deploy-preview-514--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.

@hami-robot hami-robot Bot added the size/L label Jun 28, 2026
@coderabbitai

coderabbitai Bot commented Jun 28, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 4592c089-3b9f-45c5-ad82-730b45d22aa6

📥 Commits

Reviewing files that changed from the base of the PR and between a3e36d4 and 6840b29.

⛔ Files ignored due to path filters (5)
  • static/img/ecosystem/cozystack.svg is excluded by !**/*.svg
  • static/img/ecosystem/kai-scheduler.png is excluded by !**/*.png
  • static/img/ecosystem/olares.png is excluded by !**/*.png
  • static/img/ecosystem/qwen.svg is excluded by !**/*.svg
  • static/img/ecosystem/ray.svg is excluded by !**/*.svg
📒 Files selected for processing (4)
  • src/components/ecosystemList.js
  • src/data/ecosystem.json
  • src/pages/index.js
  • src/pages/styles.module.css

📝 Walkthrough

Walkthrough

Adds src/data/ecosystem.json with six ecosystem entries, introduces an EcosystemList React component with i18n-aware logo/name rendering, integrates the "Works with HAMi" section into the home page, adds Qwen and KAI Scheduler to hero ecosystem lists, and widens the scheduler grid from 4 to 5 columns.

Changes

Ecosystem Section

Layer / File(s) Summary
Ecosystem data and EcosystemList component
src/data/ecosystem.json, src/components/ecosystemList.js
Adds six ecosystem objects (name, nameZh, logo, website) and implements EcosystemList, which maps entries to logo/name cards with locale selection and optional anchor wrapping.
Home page integration and grid layout
src/pages/index.js, src/pages/styles.module.css
Imports EcosystemList, inserts Qwen and KAI Scheduler into hero ecosystem lists, renders the new "Works with HAMi" section, and increases .schedulerEcosystemGrid column counts across all breakpoints.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐇 Six friends joined the HAMi crew,
Logos and names, in English and 中文 too,
A grid grew wider, one column more,
The ecosystem page now opens a door,
Hopping along with each lazy-loaded sprite~

🚥 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%. 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 matches the main change: adding the EcosystemList component and new ecosystem data.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@mesutoezdil

Copy link
Copy Markdown
Contributor

is line-for-line the same as adoptersList.js, only the data import and fallback logo prefix differ. consider passing data and imgPrefix as props to the existing component instead of a new file.

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