Skip to content

Conversation

@byronwang2005
Copy link
Contributor

@byronwang2005 byronwang2005 commented Dec 3, 2025

📝 Pull Request Template

1. Related Issue

Closes #216 #196

2. Type of Change (select one)

Type of Change: New Feature

3. Description

  • Attempt to reduce the error reports when obtaining stock data.
  • Reorganize the layout of the homepage.
    -- Add TradingView plugin to display the index information.
    -- Hide certain parameters helps to avoid congestion.
    -- When the proxy is unavailable or the index data is unavailable, the partial table is hidden.
  • Add TradingView plugin to display the stock information.
  • Compatible with the color settings in the settings menu.

4. Testing

  • I have tested this locally.
  • I have updated or added relevant tests.

5. Checklist

@byronwang2005
Copy link
Contributor Author

byronwang2005 commented Dec 4, 2025

截屏2025-12-04 13 38 51

@byronwang2005
Copy link
Contributor Author

Known issue:

  • Homepage plugin does not support red/green color switching.
  • Watchlist uses Yfinance data while Stock uses TV data, potentially causing conflicts.

@byronwang2005 byronwang2005 requested a review from vcfgv December 4, 2025 06:47
@byronwang2005
Copy link
Contributor Author

home5

@vcfgv
Copy link
Collaborator

vcfgv commented Dec 5, 2025

Possibly related issue #196 #216

@byronwang2005
Copy link
Contributor Author

byronwang2005 commented Dec 5, 2025

Hi! I’ve completed the encapsulation. The following interface is exposed (updated):

tradingview-ticker-tape.tsx

  • symbols: readonly string[]
  • theme?: 'light' | 'dark'(default 'light')
  • locale: "en"

tradingview-advanced-chart.tsx

  • ticker: string
  • interval?: string(default "D")
  • minHeight?: number(default 420)
  • theme?: 'light' | 'dark'(default 'light')
  • colors?: { upColor: string; downColor: string }
  • locale: "en"
  • timezone: "UTC"

@byronwang2005

This comment was marked as outdated.

@byronwang2005

This comment was marked as outdated.

@DigHuang DigHuang merged commit 4742fae into ValueCell-ai:main Dec 15, 2025
6 checks passed
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.

Still can't get the stock data

3 participants