penguin-teal / gnome-openweather

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

Launches, but drop down stopped working #5

Closed joekm closed 9 months ago

joekm commented 10 months ago

Got this yesterday from the Gnome Extensions site and it seemed to be working fine. So, I uninstalled the old version by toppk I got directly from git. This morning, it will launch but the drop down doesn't work at all.

penguin-teal commented 10 months ago

Thanks for the bug report, I've been able to reproduce the behavior and I don't believe it is related to Toppk's extension. If the extension starts when your PC has no Internet, it fails to start. I will fix this for v128. Would you wanna try going to the extensions app, disabling, and re-enabling (make sure you currently have Internet) and verify it then works?

joekm commented 10 months ago

Disabling and re-enabling has no impact. The computer I'm running is my desktop PC that connected via ethernet cable so, internet starts right away. On the other hand, I do run firewalld, which takes a couple seconds to start and I also have NVMe memory so my computer goes from GDM to desktop really fast.

I was wondering if deleting the old version (toppk) impacted a configuration file your version needed, was going to try removing and re-installing. I'll hold off for now though.

matteopaolucci commented 10 months ago

Disabling and re-enabling has no impact. The computer I'm running is my desktop PC that connected via ethernet cable so, internet starts right away. On the other hand, I do run firewalld, which takes a couple seconds to start and I also have NVMe memory so my computer goes from GDM to desktop really fast.

I was wondering if deleting the old version (toppk) impacted a configuration file your version needed, was going to try removing and re-installing. I'll hold off for now though.

Same problem here. After installation it worked fine, now after one reboot the drop-down menu isn't working. I have already tried to disable and re-enable the extension, but it doesn't work.

penguin-teal commented 10 months ago

@joekm @matteopaolucci Can you verify you are using the GNOME Extensions Website version and not the version currently on GitHub (the latest git version is not stable). If so can you open the "Settings" window via GNOME Extension app? Also does the extension show a temperature in the panel or just "..."? Sorry for all the questions but I haven't been able to reproduce the issue with an Internet connection on either my NVMe laptop or my hardrive desktop (although both use Wi-Fi).

joekm commented 10 months ago

Yes, I used the Gnome extensions website. Looking through the settings, I did notice I was getting an error message regarding the lat, long coordinates. I get this message even if I use the cords provided by your extension.

On Wed, Jan 31, 2024, 9:58 AM penguin-teal @.***> wrote:

@joekm https://github.com/joekm @matteopaolucci https://github.com/matteopaolucci Can you verify you are using the GNOME Extensions Website version and not the version currently on GitHub (the latest git version is not stable). If so can you open the "Settings" window via GNOME Extension app? Also does the extension show a temperature in the panel or just "..."? Sorry for all the questions but I haven't been able to reproduce the issue with an Internet connection on either my NVMe laptop or my hardrive desktop (although both use Wi-Fi).

— Reply to this email directly, view it on GitHub https://github.com/penguin-teal/gnome-openweather/issues/5#issuecomment-1919275661, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA76BAKGWP3ID363OFCMBXTYRJLZLAVCNFSM6AAAAABCRFJBM6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJZGI3TKNRWGE . You are receiving this because you were mentioned.Message ID: @.***>

penguin-teal commented 10 months ago

@joekm that error message is issue #2, I've fixed that for the next release. In the panel, do you see a cloud/sun icon and the temperature? or just a refresh icon and "..."?

joekm commented 10 months ago

Yes, I see the temperature and icon in the picture. Screen grab posted. (and I can get into setting via the Gnome Extension app). image

The bugs will ultimately get squashed. This is a very nice refinement to what is the best weather app for Gnome. :)

joekm commented 10 months ago

Looks to be working, thanks :)

penguin-teal commented 10 months ago

Thanks, I'm glad it got fixed!

joekm commented 10 months ago

Alas, the problem has returned. Shows weather but drop down is not working.

jimibananas commented 9 months ago

Hello - I am having the same problem. Drop down is not working. I am running Solus 4.5 Gnome and this seemed to start for me after the weekly updates applied on Friday Feb 09. Thank you.

penguin-teal commented 9 months ago

I am going to release an update soon (I've been busy) that'll have a place where you can copy your preferences as JSON onto your clipboard, and I'll try to use that to diagnose the issue since I still haven't been able to recreate it on multiple devices.

washran commented 9 months ago

I was able to view dropdown on Manjaro with Gnome 45.2 when I updated to gnome 45.3 the dropdown stopped working. When you hover over the weather top bar, it highlights, that's all. Not sure I'm explaining the bug right? Only way to edit the extension is to use the gnome-extension app of any distro.

jimibananas commented 9 months ago

Yes, that is exactly the same as what I am experiencing. I also have Gnome 45.3 which Solus must have pushed last Friday because it worked fine before that. Thanks for looking into this. I love the extension. Update- I looked at the Solus blog and they pushed Gnome 45.3 back on January 19 and I know the extension was working then. Here is a link to the sync updates for last week, I think something in here stopped it. You would know better than me. https://discuss.getsol.us/d/10310-sync-updates-for-week-6-2024

penguin-teal commented 9 months ago

Thanks for your guys' comments, that is very useful. Both my machines I've tested on are GNOME 45.4, so this might be a GNOME 45.3 issue (not sure why it'd be, but it's a lead!). When I have time I will use a VM and try it from there, however I am still very busy so I apologize it'll probably take up to a week to fix. (Also if anyone sees who is willing to I am definitely open to pull requests!)

jimibananas commented 9 months ago

Thank you! I might be able to set up a test machine for this in a week or so for pull requests. I'll let you know.

washran commented 9 months ago

Thanks for your guys' comments, that is very useful. Both my machines I've tested on are GNOME 45.4, so this might be a GNOME 45.3 issue (not sure why it'd be, but it's a lead!). When I have time I will use a VM and try it from there, however I am still very busy so I apologize it'll probably take up to a week to fix. (Also if anyone sees who is willing to I am definitely open to pull requests!)

I updated gnome to 45.4. Openweather is working as intended.

jimibananas commented 9 months ago

Success! I have a laptop with Fedora 39 and GNOME 45.4. Openweather is working great. I'll probably have to wait for Solus OS to update GNOME from 45.3 to 45.4 but that's OK. Thanks for looking at this!

penguin-teal commented 9 months ago

I think v129 fixes this. If you are running v129 (go to extension settings > About > OpenWeather Refined Version) and it still does not work go to About > Copy Settings JSON and click "Copy" and paste it here.

jimibananas commented 9 months ago

Hi... In Fedora 39 v128 is working great. Under Solus it still doesn't work. Solus is an independent distro and not based on Debian, Arch, etc. so I think something happened to one of the dependencies during one of the weekly updates. Installing from GNOME Extensions installs v128. Will you be updating that to make v129 available under shell version 45? I can try installing v129 from source. Will the git pull 128 or 129? I can try it but I think the problem might be Solus now.

penguin-teal commented 9 months ago

Hi @jimibananas, thanks for the comment. If you wait a few days it will come out on GNOME Extensions; there is one guy who has to manually review every update I post on there! The version on GitHub right now is v129, so if you want to manually install from here then this would be the latest version, however it might just be easier to wait for GNOME Extensions to release the update.

washran commented 9 months ago

I installed v129 and can confirm it works. I used your git repo. @penguin-teal thumbs up.

jimibananas commented 9 months ago

Hi @penguin-teal - Thank you for looking at this! I'll wait for the update to arrive on GNOME Extensions.

UPDATE: I noticed that v129 was published to GNOME Extensions. I installed it on my Solus machine and it working perfectly! Thank you!

joekm commented 9 months ago

Yup, I split my time between Gnome and Hyprland but, it's definitely working and has been for a bit now. Thanks keeping the best weather app on Linux alive :)