Skip to content

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

Closed
stefanolomo wants to merge 17 commits into
slowlydev:mainfrom
stefanolomo:main
Closed

Feat: Live standings (Jolpica), environment variables standardization, and weather map improvements#313
stefanolomo wants to merge 17 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
@slowlydev

Copy link
Copy Markdown
Owner

hey @stefanolomo thanks for opening this PR

I looked into your rainviewer and signalrcore fixes, they look pretty decent.

But I can't merge this PR as is as I saw you started developing some other features in your fork which I would like to avoid merging. (looking forward to your future changes as well)

So I ported some of your fixes in the develop branch and they will be included in the #315 release. I also mentiond your changes in the release notes.

but for now I will close this PR :)

@slowlydev slowlydev closed this Jun 6, 2026
@stefanolomo

Copy link
Copy Markdown
Author

Cool! thanks.

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