refactor: components and i18n #63
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Replace cloudflare/wrangler-action with direct npx opennextjs-cloudflare deploy command for both production and staging environments. π€ Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
β¦and CommunityLinks components - Create new ResourceLinks component for download, changelog, pricing, MCP, and issue tracker links - Refactor CommunityLinks to use i18n translations for all link titles and descriptions - Update LinkCard to handle null/empty href with grayed-out placeholder rendering - Simplify LinkCardGrid to display all links including empty ones as placeholders - Remove hasAnyPlatformUrl helper from PlatformLinks (handled by LinkCard now) - Add title props to ComparisonTable columns for tooltip support - Update all product detail pages (CLI, Extension, IDE, Model Provider) to use new link components - Update all 14 locales with new communityLinks and resourceLinks translation keys - Update component relationship documentation - Format code and fix JSON formatting issues Generated with Claude Code (https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
β¦tyLinks usage - Remove redundant communityLinks array from model-providers and vendor pages - Remove explicit title and links props from CommunityLinks component - Add index signature to ManifestResourceUrls for dynamic resource keys CommunityLinks component now handles link configuration internally, reducing duplication across pages. π€ Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Fix pages/articles.json structure to match de - Fix pages/comparison.json structure to match de - Fix pages/home.json structure to match de - Add vendorDetail.community.blog.* keys to es, pt, ru, tr - Add modelProviderDetail.community.blog.* keys to fr, id, ja π€ Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Refine LinkCard urls prop type from Record<string, unknown> to Record<string, string | null> for better type safety. π€ Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Add new translation keys for multiple components to support full internationalization: - Breadcrumb component (home label) - PlatformLinks component (title and AI platform descriptions) - ProductHero component (vendor, version, license, stars, platforms, type, link labels) - VendorProducts component (title) - VendorModels component (title) - GitHubStarHistory component (loading, title, description, stars, fallback note) π€ Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
β¦spaces Update all product and navigation components to use translation hooks: - Breadcrumb: Add home translation key - GitHubStarHistory: Move all hardcoded strings to translation namespace - PlatformLinks: Internalize links config and add translations - ProductHero: Remove labels prop, use translation hook, simplify type handling - VendorProducts: Remove title and locale props, use translation hook - VendorModels: Remove title and locale props, use translation hook π€ Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Update product detail pages to remove props that are now internalized in components: - Remove locale prop from VendorProducts and VendorModels - Remove redundant title/links props from PlatformLinks - Remove labels prop from ProductHero, use typeValue prop instead - Simplify platform links configuration π€ Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Move npm run check after npm run test:ci in pre-commit - Add npm run type-check to pre-commit - Remove pre-push hook (δΈειθ¦ pre-push check) π€ Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Preview deployment
Commit: |
Preview deployment cleaned upThe preview deployment for PR #63 has been removed. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Type of Change
Related Issue
Closes #(issue number)
Changes Made
Metadata Contribution Checklist
Only complete this section if adding/updating manifest files:
manifests/ides/,manifests/clis/, etc.)For model contributions, additionally:
Testing
npm run build)npm run validate:manifests)npm run validate:urls)npm run lint)npm run spell)Screenshots (if applicable)
Additional Notes
Reviewer Checklist