Skip to content

fix: resolve product icon rendering issue and add missing port display in domain URLs#135

Merged
slievrly merged 20 commits intohigress-group:mainfrom
learnerjohn:himarket_202512
Dec 25, 2025
Merged

fix: resolve product icon rendering issue and add missing port display in domain URLs#135
slievrly merged 20 commits intohigress-group:mainfrom
learnerjohn:himarket_202512

Conversation

@learnerjohn
Copy link
Copy Markdown
Collaborator

@learnerjohn learnerjohn commented Dec 23, 2025 •
edited
Loading

📝 Description

  • Fix product icons not displaying on card pages
  • Fix missing port information in product route URLs
  • Standardize code comments

🔗 Related Issues

Fix #134

✅ Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring (no functional changes)
  • Performance improvement
  • Build/CI configuration change
  • Other (please describe):

🧪 Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing completed
  • All tests pass locally

📋 Checklist

  • Code has been formatted (mvn spotless:apply for backend, npm run lint:fix for frontend)
  • Code is self-reviewed
  • Comments added for complex code
  • Documentation updated (if applicable)
  • No breaking changes (or migration guide provided)
  • All CI checks pass

📊 Test Coverage

📚 Additional Notes

@learnerjohn
refactor: rename project package to himarket
f2cd523 
Change-Id: I961a47e80c0b30ba94befcf786cd68eb922bd171
@learnerjohn
feat: support Higress gateway IP for Model and MCP
190a3af 
Change-Id: I189047b9adf7e5870413654d49706a914af1ec1b
@learnerjohn
fix: resolve issue where Model cannot use MCP in Higress
834cbfd 
Change-Id: I818a548d4239b13ad7cd57d1e100398b100c1757
@learnerjohn
docs: add user guide and update README with PR guidelines
570869e 
Change-Id: I4c648e944313a85e4da1c10fdc8d477288a10e4c
@learnerjohn
Merge remote-tracking branch 'upstream/main' into himarket_202512
750848d
@learnerjohn
fix: developer consumer config and docs update
9e6c5be 
- Auto-configure Primary Consumer for external developers
- Fix Higress MCP Path
- Remove emoji from documentation files

Change-Id: I02c85054cba5c9459c523d8d964f148d99395d7f
@learnerjohn
style: format code with spotless
061ad1e 
Change-Id: I91e7a57d42ae0e3455dee0ce690e3300d22d6a51
@learnerjohn
merge
96e2a61 
Change-Id: Ieb4d9b1d507301689aefd3f41b9730073705d288
@learnerjohn
Merge remote-tracking branch 'upstream/main' into himarket_202512
8497489
@learnerjohn
Merge remote-tracking branch 'upstream/main' into himarket_202512
1f17d1d
@learnerjohn
fix: correct branding to HiMarket and fix gateway address port handli…
07c092b 
…ng and consumer apikey authentication

Change-Id: I6be6853e5a2819fa46b292d1810fec03294bad87
@learnerjohn
Merge remote-tracking branch 'upstream/main' into himarket_202512
8c5ed3c
@learnerjohn
fix: resolve MCP tools and Product config problems, optimize higress …
ad2caf6 
…gateway URI parsing and resource identification

Change-Id: Ib95eef61fb18fb837f32ab3facdea28e3aa90a7c
@learnerjohn
merge
08447a1 
Change-Id: Ia4d77996cb15ce556529ea1ff7118cb70ae3ecb3
@learnerjohn
Merge remote-tracking branch 'upstream/main' into himarket_product
dbf19b8
@learnerjohn
fix: extend request timeout to resolve MCP listTools timeout and add …
19bee45 
…license headers

Change-Id: Id72fc1f6b1c849328379723b4eb907589d2e035a
@learnerjohn
fix: Update image source in README
9991e4c 
Change-Id: I48f5ecbf4f629545ff9eed7a7470ef7bd6bdb1b6
@learnerjohn
refactor: standardize code comments and improve consistency
7f693ec 
Change-Id: If3b0a15754c32f429c051a5bdbed6c32703dcde6
@learnerjohn
fix: resolve product icon rendering issue and add missing port displa…
2dca80b 
…y in domain URLs

Change-Id: I0d40aacee67059cefcd2c89fbf4f9946370715d3
slievrly
slievrly approved these changes Dec 25, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@slievrly slievrly left a comment

Choose a reason for hiding this comment

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

LGTM

@slievrly slievrly merged commit bdd0389 into higress-group:main Dec 25, 2025
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@slievrly slievrly slievrly approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[bug]门户前端 MCP 列表，显示产品的 logo 错误

2 participants

@learnerjohn @slievrly