Skip to content

Conversation

JosephScharnweber
Copy link

A Tidbyt app that helps dog owners determine safe walking conditions based on current weather data. Pawcast displays temperature, humidity, heat index, and provides risk assessments for walking dogs in hot weather.

@tidbyt
Copy link

tidbyt bot commented Sep 4, 2025

⚠️ The automated review process is experimental and likely has bugs. Please bear with us as we iron out the kinks and enable you to ship changes at high velocity 🚀

All Set 🎉

This change is ready to go! To merge your change, simply comment with a :shipit: (:shipit:) emoji on this pull request and our robots will take it from there.

Automated Review Successful

Magical robots have reviewed your change and it looks good to go! 🤖

Test Details
App Dir All files are in a single app directory
Modules Only safe modules, without side-effects, are loaded
Original Author The original author matches the PR author

@tidbyt-bot
Copy link

tidbyt-bot commented Sep 4, 2025

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@JosephScharnweber
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

tidbyt[bot]
tidbyt bot previously approved these changes Sep 4, 2025
tidbyt[bot]
tidbyt bot previously approved these changes Sep 4, 2025
@danielsitnik
Copy link
Collaborator

Hey @JosephScharnweber, is this still a work in progress?

I'm asking because you have some config.get calls in the code, but you don't have the get_schema() function defined, so these calls aren't really doing anything.

From what I understood, the app doesn't use any APIs to get the weather info, the user should input the temperature and humidity in the configuration.. but since you don't have get_schema defined, it's always using the default value.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants