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

Enable Gadgetbridge support. #680

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

KamaleiZestri
Copy link

To resolve #549

This makes it so that Forecastie sends its weather data to Gadgetbridge upon any data updates from OpenWeather. This feature has to be enabled via settings to function.

Due to how Gadgetbridge processes some of the weather data, some of the accuracy of Forecastie is lost when transferred. Most further improvements to accuracy or data sent would have to come from the Gadgetbridge side first.

@Sokoloft
Copy link

Hello,

I hope this gets implemented. As I just made feature requests for this exact feature. I'll look into building Forecastie from source. I would love to have this functionality with my Pinetime.


Intent intent = new Intent();
intent.setAction("de.kaffeemitkoffein.broadcast.WEATHERDATA");
intent.setPackage("nodomain.freeyourgadget.gadgetbridge");

Choose a reason for hiding this comment

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

Wondering if this should be configurable, since there are a few different flavors now (eg. nightly, bangle.js).

Copy link
Author

@KamaleiZestri KamaleiZestri Aug 14, 2022

Choose a reason for hiding this comment

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

That makes sense also for future compatibility. It is done.

@robinpaulson
Copy link
Contributor

There is not currently a maintainer for Forecastie, this will not be merged until that changes. Perhaps @martykan is interested in hearing from potential maintainers?

@Sokoloft
Copy link

Sokoloft commented Jul 7, 2023

Just to update. This fork does indeed work for the pinetime now that it supports weather. I have had no crashing and the app has been stable. 👍 Thank you for the fork @KamaleiZestri

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.

Support for GadgetBridge Notifications
4 participants