Skip to content

Feat: Live standings (Jolpica), environment variables standardization, and weather map improvements#313

Open
stefanolomo wants to merge 8 commits into
slowlydev:mainfrom
stefanolomo:main
Open

Feat: Live standings (Jolpica), environment variables standardization, and weather map improvements#313
stefanolomo wants to merge 8 commits into
slowlydev:mainfrom
stefanolomo:main

Conversation

@stefanolomo
Copy link
Copy Markdown

This PR introduces general improvements to the project's environment configurations, migrates the driver and constructor standings panel to fetch live data from the Jolpica (Ergast) API, and implements critical lifecycle and visibility fixes in the weather map radar.

Changes Introduced

Configuration and Environment (Chore)

  • Standardized environment variables documentation by adding detailed .env.example files at the project root and within each submodule (api/, realtime/, and simulator/).
  • Updated TypeScript module resolution in dashboard/tsconfig.json from "node" to "bundler".
  • Ignored local .txt and .sh file patterns in .gitignore.

Live Championship Standings (Feat)

  • Removed the static mock standings store.
  • Implemented dynamic queries to the Jolpica API endpoints (driverstandings.json and constructorstandings.json).
  • Added SVG logos for new constructors Audi and Cadillac to dashboard/public/team-logos/.
  • Redesigned loading skeletons (DriverSkeletonItem and TeamSkeletonItem) to match the new layout structures.

Weather Map Optimizations (Fix)

  • Fixed a double slash issue in the Rainviewer tile source URL.
  • Added map cleanup logic (mapInstance.remove()) and an isMounted mount check to the useEffect hook to prevent memory leaks and visual bugs on fast page transitions.
  • Optimized radar layer rendering using layout visibility property toggling.
  • Changed default initial zoom to 6 to provide a better regional view of incoming rain.

@stefanolomo stefanolomo requested a review from slowlydev as a code owner May 25, 2026 03:40
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.

1 participant