![]() |
|
[RELEASE] Weather Plus Addon - Printable Version +- XBMC Community Forum (http://forum.xbmc.org) +-- Forum: Help and Support (/forumdisplay.php?fid=33) +--- Forum: Add-ons Help and Support (/forumdisplay.php?fid=27) +---- Forum: Weather Add-ons (/forumdisplay.php?fid=155) +---- Thread: [RELEASE] Weather Plus Addon (/showthread.php?tid=95329) |
- macf1an - 2011-11-19 10:36 Just got 2.5.0 and unless you manually go with the location selector on every one of the three options in settings - you get "location not selected" message. Why doesn't it get the proper locations from the global area codes as before? And the top-of-the-screen temperature indicator from confluence is still gone.. - m4x1m - 2011-11-19 13:47 macf1an Wrote:Just got 2.5.0 and unless you manually go with the location selector on every one of the three options in settings - you get "location not selected" message. Same here. Pre-Eden PVR version doesn't have a git number. Git number is UNKNOWN and this break the script. - brightsr - 2011-11-19 14:40 olympia Wrote:Is there any reason why you are not requesting this to be included in the official repo? I'll do when pre-eden issue is gone. - brightsr - 2011-11-19 15:17 macf1an Wrote:Just got 2.5.0 and unless you manually go with the location selector on every one of the three options in settings - you get "location not selected" message. You prefer to the past way. I was thinking all options and area codes should be in the addon's settings as the built-in weather api has been gone. But if users don't like it and want to go back, I'll make a version for Dharma only. - brightsr - 2011-11-19 15:20 m4x1m Wrote:Same here. Oh, I didn't consider PVR version. Is the build version like "PRE-11.0 Git:UNKNOWN"? - Big_Noid - 2011-11-19 15:26 Just reporting in that 2.5 works flawlessly for me on eden-pre. Only one very very very minor thing: current condition capitalizes also the first letter in the second word. I know accuweather provides this in all capitals so there must be some conversion in the script to lowercase. - m4x1m - 2011-11-19 15:30 brightsr Wrote:Oh, I didn't consider PVR version. unfortunately yes! I bypassed the check, and I get the "location not selected" message, but latest PVR build is compiled on November 12, I think I'll have to wait a new build to have weather work correctly. - olympia - 2011-11-19 15:34 brightsr Wrote:I'll do when pre-eden issue is gone. I don't think there is a pre-eden issue... It work flawless for me on pre-eden and others reported this as well above. The rest is user error. - brightsr - 2011-11-19 15:38 Big_Noid Wrote:Only one very very very minor thing: current condition capitalizes also the first letter in the second word. I know accuweather provides this in all capitals so there must be some conversion in the script to lowercase. hmm.. strange.. I already made it to convert current condition to titlecase. (only the first letter is uppercase) - brightsr - 2011-11-19 16:06 olympia Wrote:I don't think there is a pre-eden issue... It work flawless for me on pre-eden and others reported this as well above. The rest is user error. I'd like to see if there's no issue until tonight, and then I'll request. |