Skip to content

Conversation

@Gaubee
Copy link
Contributor

@Gaubee Gaubee commented Jan 16, 2026

变更说明

删除了 WalletAddressPortfolioFromProvider 组件及其相关的 Storybook stories:

删除的内容

  1. WalletAddressPortfolioFromProvider 组件

    • 只被 Storybook stories 使用
    • 需要完整的 ChainProvider 基础设施
    • 没有被任何实际业务页面使用
  2. Real Data stories (需要网络请求)

    • RealDataBfmeta, RealDataEthereum, RealDataBitcoin, RealDataTron, RealDataBinance
    • 这些在 CI 环境中可能不稳定
  3. Compare Providers stories (复杂基础设施)

    • CompareProvidersBfmeta, CompareProvidersEthereum
    • 需要动态注入配置,逻辑复杂
  4. 相关基础设施代码

    • ChainConfigProvider, TokenIconProviderWrapper
    • DynamicCompareConfigInjector, DynamicCompareProvidersGrid
    • 各种 decorators

保留的内容

  • 所有使用 mock 数据的 UI 测试 stories
  • Default, Loading, Empty - 基础状态
  • 各链的 NormalDataFallbackWarning stories
  • PartialFallback, EmptyButSupported - 边界情况

效果

  • 文件从 998 行 减少到 329 行 (-67%)
  • 删除了 738 行代码
  • 移除了对网络请求的依赖
  • CI 构建更加稳定

- Remove WalletAddressPortfolioFromProvider component (only used by stories)
- Remove all Real Data stories that require network requests
- Remove all Compare Providers stories with complex infrastructure
- Keep only mock-based UI tests for stable CI
- Reduce stories file from 998 lines to 329 lines
@Gaubee Gaubee force-pushed the refactor/simplify-portfolio-stories branch from 9adb2a8 to 61309af Compare January 16, 2026 08:59
@Gaubee Gaubee merged commit 728f022 into main Jan 16, 2026
5 checks passed
@Gaubee Gaubee deleted the refactor/simplify-portfolio-stories branch January 16, 2026 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants