feat: additional validator info - #430
Conversation
🦋 Changeset detectedLatest commit: bc6bf64 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull Request Overview
This pull request enhances the validator selection UI by adding new metadata fields, significantly expands French translations, and updates dependencies. The changes improve internationalization support and provide more detailed validator information for users.
Key changes:
- Added support for displaying
subnetName,marketCap, andtokenSymbolin validator metadata - Expanded French translations with new UI strings, campaign descriptions, and action labels
- Updated dependencies including
@biomejs/biometo 2.2.0 and@stakekit/api-hooksto 0.0.109
Reviewed Changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| packages/widget/src/components/molecules/select-validator/meta-info.tsx | Added support for new validator metadata fields (subnetName, marketCap, tokenSymbol) |
| packages/widget/src/components/molecules/select-validator/select-validator-list.tsx | Passed new metadata props to validator components and added debug console.log |
| packages/widget/src/translation/French/translations.json | Significantly expanded French translations for UI elements, campaigns, and actions |
| packages/widget/src/translation/English/translations.json | Added English translations for new validator metadata fields |
| packages/widget/tests/utils/setup.ts | Updated TypeScript directive from @ts-ignore to @ts-expect-error |
| packages/widget/package.json | Updated @stakekit/api-hooks dependency to version 0.0.109 |
| package.json | Updated @biomejs/biome dependency to version 2.2.0 |
| biome.json | Disabled additional linting rules for project needs |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| const metaInfo = useMetaInfo(props); | ||
|
|
||
| console.log({ metaInfo }); | ||
|
|
There was a problem hiding this comment.
Debug console.log statement should be removed before merging to production. This appears to be leftover debugging code that will clutter the console output.
628a4e3 to
0da322f
Compare
0da322f to
26fb10f
Compare
This pull request introduces several improvements and new features to the validator selection UI, updates dependencies, and significantly expands French translations for the widget. The main highlights are the addition of new validator metadata fields, enhanced internationalization, and dependency updates for both core and widget packages.
Validator Metadata Enhancements:
subnetName,marketCap, andtokenSymbolas part of validator metadata in theuseMetaInfohook and related components. This includes updating the prop types, logic, and rendering. [1] [2] [3] [4] [5] [6] [7]Internationalization (i18n):
Dependency Updates:
@biomejs/biomefrom version 2.1.2 to 2.2.0 in bothpackage.jsonandpnpm-lock.yaml. [1] [2]@stakekit/api-hooksfrom version 0.0.108 to 0.0.109 in bothpackages/widget/package.jsonandpnpm-lock.yaml. [1] [2]Development and Linting Configuration:
biome.jsonto turn off several linting rules (useIterableCallbackReturn,useUniqueElementIds,useIndexOf) to better match project needs.Testing Utilities:
@ts-expect-errorfor better clarity and intent.