penguin-teal / gnome-openweather

A GNOME Shell extension to show the weather of any location on Earth.
GNU General Public License v3.0
52 stars 17 forks source link

Forecast in pop-up is blank in GNOME 45 #18

Closed S0litaire closed 6 months ago

S0litaire commented 6 months ago

I think recent update has broken something in the widget.

The Forecast drop down is empty.

Ubuntu : 23.10 Firefox : 124.0.1

Settings JSON

{"app-version":"132","git-version":"131.r18.g8cddeed","gnome-version":"45.2","appid":"'b4cbc29034cd3db6dea22928c930865b'","use-default-owm-key":"false","actual-city":"0","prefs-default-width":"700","hi-contrast":"'none'","sun-in-panel-first":"true","has-run":"true","days-forecast":"3","refresh-interval-current":"900","location-text-length":"3","prefs-default-height":"960","delay-ext-init":"4","locs":"[(0, '<NAME>', 0, '<PLACE>')]","center-forecast":"false","my-loc-prov":"'ipinfoio'","show-sunsetrise-in-panel":"true","position-index":"4","geolocation-provider":"'openstreetmaps'","show-comment-in-forecast":"true","use-system-icons":"true","simplify-degrees":"false","wind-direction":"true","pressure-unit":"'bar'","disable-forecast":"false","clock-format":"'24h'","menu-alignment":"100.0","position-in-panel":"'center'","wind-speed-unit":"'mph'","show-comment-in-panel":"true"}

Here is what it looks like on my system:

Screenshot from 2024-03-31 21-41-26

mcwall1064 commented 6 months ago

Exact same issue... Fedora 39 Gnome 45.5 Firefox 124.0.1

penguin-teal commented 6 months ago

I assume the issue must be that the update for GNOME 46 broke something on GNOME 45... Thanks for the report.

penguin-teal commented 6 months ago

In trying to fix it for GNOME 46 it broke GNOME 45! Now both versions work with the new commit.

pawel-rh commented 6 months ago

Unfortunately that is still an open issue

{"app-version":"132","git-version":"131.r18.g8cddeed","gnome-version":"45.2","actual-city":"0","translate-condition":"false","sun-in-panel-first":"false","has-run":"true","days-forecast":"5","decimal-places":"1","delay-ext-init":"3","locs":"[(0, '', 0, '')]","expand-forecast":"true","show-sunsetrise-in-panel":"false","simplify-degrees":"true","wind-direction":"true","pressure-unit":"'mbar'","disable-forecast":"false","show-text-in-panel":"false","loc-refresh-interval":"1440.0","show-comment-in-panel":"false"}

image

S0litaire commented 6 months ago

just got around to updating and yes it's still blank json :

{"app-version":"132","git-version":"132.r0.g5c5eac4","gnome-version":"45.2","appid":"'b4cbc29034cd3db6dea22928c930865b'","use-default-owm-key":"false","actual-city":"0","prefs-default-width":"977","hi-contrast":"'white'","has-run":"true","days-forecast":"5","prefs-default-height":"997","locs":"[(0, '<NAME>', 0, '<PLACE>')]","center-forecast":"true","expand-forecast":"true","my-loc-prov":"'ipinfoio'","show-sunsetrise-in-panel":"true","position-index":"1","geolocation-provider":"'openstreetmaps'","show-comment-in-forecast":"true","simplify-degrees":"false","wind-direction":"true","pressure-unit":"'mbar'","disable-forecast":"false","clock-format":"'24h'","menu-alignment":"87.0","position-in-panel":"'center'","wind-speed-unit":"'mph'","show-comment-in-panel":"true"}

penguin-teal commented 6 months ago

@S0litaire @pawel-rh Sorry to clarify the latest development commit fixed it; there has been no release since then (I still want to fix something else before I make another release). When v133 comes out it will ship with the bug fix. (Commit 952dc54 is newer than both of the versions in your guys' git-version.)