Skip to content

Conversation

JasperX777
Copy link

@JasperX777 JasperX777 commented Aug 9, 2025

JasperX777 JasperX777 /jasper/fix-1887-weather-dates → Lissy93/dashy Commits: 2 | Files Changed: 2 | Additions: 4 Label Unchecked Tasks Powered by Pull Request Badge

Category:

Bugfix

Overview

Fix weather widget API endpoint logic that was hardcoded for OpenWeatherMap.
Added support for WeatherAPI by adjusting endpoint construction, query parameters, and data parsing logic.

Issue Number (if applicable) #1887 Weather forecast showing same day multiple times

New Vars (if applicable)

If you've added any new build scripts, environmental variables, config file options, dependency or devDependency, please outline here

Screenshot (if applicable)

截屏2025-08-09 下午3 30 54

Code Quality Checklist (Please complete)

  • All changes are backwards compatible
  • All lint checks and tests are passing
  • There are no (new) build warnings or errors
  • (If a new config option is added) Attribute is outlined in the schema and documented
  • (If a new dependency is added) Package is essential, and has been checked out for security or performance
  • (If significant change) Bumps version in package.json

@JasperX777 JasperX777 requested a review from Lissy93 as a code owner August 9, 2025 03:32
Copy link

netlify bot commented Aug 9, 2025

Deploy Preview for dashy-dev ready!

Name Link
🔨 Latest commit 2dddbba
🔍 Latest deploy log https://app.netlify.com/projects/dashy-dev/deploys/6896c1361a3a3e0008fde0ea
😎 Deploy Preview https://deploy-preview-1900--dashy-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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