terminal-for-fitbit icon indicating copy to clipboard operation
terminal-for-fitbit copied to clipboard

Published 20 hours ago • verified publisher icon jdhartley

feature request - add weather as screen data

Open muymanwell opened this issue 3 years ago • 6 comments

it would be wonderful if weather could be added as an option (temp/current weather type) for the face

muymanwell avatar Feb 06 '21 18:02 muymanwell

Hi @muymanwell, thanks for the suggestion! I'd love to bring this feature the Terminal in a future update.

jdhartley avatar Feb 12 '21 01:02 jdhartley

Hi, @jdhartley, are you still maintaining this project? I'd be happy to open a PR to add a weather dataline if it's something you'd be willing to publish.

jensenak avatar Sep 20 '21 21:09 jensenak

hi @jensenak, yes I am still here to publish updates. I have not worked on this project much since the latest fitbit SDK fracture, but am still wearing this clock face every day!

I believe that the weather data can be pulled straight from the Fitbit SDK now*, so this change is hopefully fairly easy to add in.

if you want to open a PR adding this in to the sdk5 branch, I would love to test and get a build going! https://dev.fitbit.com/build/reference/companion-api/weather/

*note that this would mean this feature is only available on Sense and Versa 3 devices. as of now, i am not too interested in updating the sdk4 branch with HTTP requests

jdhartley avatar Sep 22 '21 00:09 jdhartley

Sounds like a plan. I have a Versa 3 and that companion api looks pretty easy to work with. I'll check out the sdk5 branch. Might be a couple days on the PR, since I've got other stuff in progress as well.

jensenak avatar Sep 22 '21 00:09 jensenak

Forgot to ask, does the name WTHR sound okay for the data line? Or maybe TEMP?

jensenak avatar Sep 22 '21 00:09 jensenak

If we are just showing the temperature, lets go with TEMP, e.g.

[TEMP] 70º

Though as I am typing, I doubt think we have a degrees symbol in the fitfont folders right now. We could generate those (I think the readme talks about how), but if we want to just put 6 Celsius or 42 Fahrenheit that is ok with me. If you open a PR I can find some time to help generate those glyphs and maybe it can be an option for Full Unit / Degrees Symbol.

[TEMP] 31 Celsius
[TEMP] -3 Fahrenheit

For WTHR I would love to find a way to get WeatherCondition in there too like 70º Rainy, but looking at the enum list, oof... If you have thoughts on how to fit PartlySunnyWithThunderstormsDay in a dataline, very open to ideas 😳

jdhartley avatar Sep 22 '21 00:09 jdhartley