Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Schedule Criteria: weather "clear sky" condition does not match #2740

Merged
merged 5 commits into from
Oct 2, 2024

Conversation

nadzpogi
Copy link
Contributor

relates to xibosignage/xibo#3487

  • Modified metric IDs to better align with JSON responses from the OWM API.
  • Added a prefix to avoid conflicts with other potential weather schedule criteria.
  • Removed metrics that are not present in OpenWeather API responses.

relates to xibosignage/xibo#3487

- Modified metric IDs to better align with JSON responses from the OWM API.
- Added a prefix to avoid conflicts with other potential weather schedule criteria.
- Removed metrics that are not present in OpenWeather API responses.
@nadzpogi nadzpogi self-assigned this Sep 18, 2024
relates to xibosignage/xibo#3487

- Added a prefix to the weather XMDS response.
- Restored Wind Direction with fewer options after it was previously removed.
PeterMis
PeterMis previously approved these changes Sep 19, 2024
PeterMis
PeterMis previously approved these changes Sep 20, 2024
Copy link
Member

@dasgarner dasgarner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The OWM prefix doesn't make sense - criteria should be service agnostic so a prefix of weather_ would make more sense.

relates to xibosignage/xibo#3487

- Updated the weather schedule criteria's and XMDS getWeather's prefix.
dasgarner
dasgarner previously approved these changes Oct 1, 2024
relates to xibosignage/xibo#3487

- Fixed the formatting of weather condition values.
- Set the `type` criteria as the prefix for the metric's value
@ifarzana
Copy link
Contributor

ifarzana commented Oct 2, 2024

TEST: /Schedule END

Copy link

github-actions bot commented Oct 2, 2024

@nadzpogi nadzpogi merged commit 93c5cd1 into develop Oct 2, 2024
2 of 3 checks passed
@nadzpogi nadzpogi deleted the bugfix/giacobini_openweather_schedule_criteria branch October 2, 2024 10:34
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.

4 participants