Skip to content

feat: Deploy React dashboard to Netlify with TypeScript fixes - #339

Open
Raakshass wants to merge 2 commits into
openclimatefix:mainfrom
Raakshass:feature/deploy-react-dashboard
Open

feat: Deploy React dashboard to Netlify with TypeScript fixes#339
Raakshass wants to merge 2 commits into
openclimatefix:mainfrom
Raakshass:feature/deploy-react-dashboard

Conversation

@Raakshass

Copy link
Copy Markdown
Contributor

Summary

Deploy the React dashboard to Netlify with TypeScript fixes and correct API integration.

🌐 Live Dashboard: https://quartz-solar-dashboard.netlify.app

Closes #153


Changes Made

1. API Integration Fix

  • Changed API URL from http://localhost:8000 to https://open.quartz.solar/forecast/
  • Wrapped request body in {site: {...}} format to match API v1 spec
  • Fixed response parsing to access data.predictions.power_kw

2. TypeScript Fixes

File Fix
tsconfig.app.json Added path aliases (@/*)
PVForecastForm.tsx Type annotations
PredictionTable.tsx Type annotations
PredicitionChart.tsx Removed unused import, optional chaining
calendar.tsx Removed unused props
App.tsx Proper useState typing

Screenshot

Screenshot 2026-02-03 234030

Checklist

- Fix API URL to use open.quartz.solar instead of localhost
- Wrap request body in {site: ...} format per API v1 spec
- Fix response parsing to use data.predictions.power_kw
- Add TypeScript type annotations to fix build errors
- Add path aliases to tsconfig.app.json for @ imports
- Remove unused imports and props

Closes openclimatefix#153
@Raakshass
Raakshass force-pushed the feature/deploy-react-dashboard branch from 7a41fec to 534ae8c Compare February 3, 2026 19:04
@Raakshass

Copy link
Copy Markdown
Contributor Author

CI Status Update

The lint-typecheck (pull_request) check passes ✅, confirming this PR's code is correct.

The lint-typecheck (pull_request_target) check fails because main has a whitespace issue on line 102 of open_meteo.py. This is a known issue with GitHub's pull_request_target trigger which runs against the base branch.

Fix: PR #340 addresses this by removing the trailing whitespace. Once #340 is merged, this PR's CI will pass.

🌐 Live Dashboard: https://quartz-solar-dashboard.netlify.app

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.

Deploy api and dashboards

1 participant