search

andrewmm / ncwunderground

iPhone Notification Center Widget for Weather Underground
8 stars 1 forks source link

Display status/error messages in a less obtrusive manner. #30

Open andrewmm opened 11 years ago

andrewmm commented 11 years ago

"It keeps telling me "update location failed". I have poor reception in the area that I live and just about every time I look at my phone I have to dismiss this annoying error. I really like your widget and appreciate your effort, I just don't understand why the app needs to tell me every time it fails to figure out my location."

andrewmm commented 11 years ago

Removed the error messages in question in 5f13f3324d36aad58f8b5537646101e6e82a8a24. Still need to think of an alternative.

roadiemoose commented 11 years ago

Possibly related to same connection issues i was having while using ISX 6. When phone is locked, there is no way to accept "Springboard wants to use my location" popup. I finally had time to test this and I'm getting "update location failed" even with ISX not installed. But i get the connection error message i posted about a few days ago only with ISX installed. I think it may have to do with the fact that your tweak requests permission for springboard to update my location every time it updates. Is there a way to make this a "remembered" option. IE once i say yes, you can use my location, stop asking?? :) I think that will solve some problems. I also think that there are issues with your tweak falling back on cellular data when wifi isn't connected. I notice that if I have wifi turned off, the "update location" error happens less often. If i unlock my phone, it takes a second for my phone to re establish the wifi connection. Maybe set a delay of a few seconds before the tweak attempts to update location??

Thanks and sorry for the delay in getting back to you again. Work has been a PitA..

andrewmm commented 11 years ago

I've found a way to make it save the location permissions and it's already coded; it will be in the next release. (It requires a private API call.)

I can probably give the connection attempt a slightly longer timeout before it fails. — Andrew MacKie-Mason

On Sat, May 11, 2013 at 9:06 AM, roadiemoose notifications@github.com wrote:

Possibly related to same connection issues i was having while using ISX 6. When phone is locked, there is no way to accept "Springboard wants to use my location" popup. I finally had time to test this and I'm getting "update location failed" even with ISX not installed. But i get the connection error message i posted about a few days ago only with ISX installed. I think it may have to do with the fact that your tweak requests permission for springboard to update my location every time it updates. Is there a way to make this a "remembered" option. IE once i say yes, you can use my location, stop asking?? :) I think that will solve some problems. I also think that there are issues with your tweak falling back on cellular data when wifi isn't connected. I notice that if I have wifi turned off, the "update location" error happens less often. If i unlock my phone, it takes a second for my phone to re establish the wifi connection. Maybe set a delay of a few seconds befo re the tweak attempts to update location??

Thanks and sorry for the delay in getting back to you again. Work has been a PitA..

Reply to this email directly or view it on GitHub: https://github.com/andrewmm/ncwunderground/issues/30#issuecomment-17760470

roadiemoose commented 11 years ago

Any idea when you might push an update?? The error pop ups are really annoying.. It's definitely an issue with ISX because I can get an update as long as the phone is locked..

andrewmm commented 11 years ago

Should be relatively soon. — Andrew MacKie-Mason

On Fri, May 17, 2013 at 3:09 AM, roadiemoose notifications@github.com wrote:

Any idea when you might push an update?? The error pop ups are really annoying.. It's definitely an issue with ISX because I can get an update as long as the phone is locked..

Reply to this email directly or view it on GitHub: https://github.com/andrewmm/ncwunderground/issues/30#issuecomment-18048825

andrewmm commented 11 years ago

Also, correct me if I'm wrong, but I believe you can selectively disable widgets from IntelliScreen X, and still have them show up in the Notification Center? — Andrew MacKie-Mason

On Fri, May 17, 2013 at 3:09 AM, roadiemoose notifications@github.com wrote:

Any idea when you might push an update?? The error pop ups are really annoying.. It's definitely an issue with ISX because I can get an update as long as the phone is locked..

Reply to this email directly or view it on GitHub: https://github.com/andrewmm/ncwunderground/issues/30#issuecomment-18048825

roadiemoose commented 11 years ago

I don't believe that to be true.. ISX "hijacks" the NC settings.. ISX features are visible in both unlocked and locked state..

andrewmm commented 11 years ago

Huh. I only had it installed for a little while for debugging, maybe I'm confusing it with another tweak.

Update should be out soon. — Andrew MacKie-Mason

On Fri, May 17, 2013 at 6:17 PM, roadiemoose notifications@github.com wrote:

I don't believe that to be true.. ISX "hijacks" the NC settings.. ISX features are visible in both unlocked and locked state..

Reply to this email directly or view it on GitHub: https://github.com/andrewmm/ncwunderground/issues/30#issuecomment-18090617

roadiemoose commented 11 years ago

Without ISX or some other tweak(Bulletin etc..) Notification Screen is not visible when the phone is locked.. So removing ISX wouldn't change anything.. The problem is your tweak needs to be able to get permission to call the server to get a location update and then get the weather info when the phone is locked. That's how I see this problem.. Agree?? When the phone is unlocked and I swipe down to show the NC screen, I get a pop up asking to give springboard location permission.. This window obviously can't pop up when the phone is locked, therefore the tweak times out trying to reach the server..

Sorry if I'm repeating myself or saying something you already know..

andrewmm commented 11 years ago

I've figured out my own solution for location permission, and it's coded up. I also just figured out (/got help with) the localization issue that was holding up the update. I'm at my brother's graduation today, but hopefully I'll be able to get everything coded and an update to Big Boss tomorrow. — Andrew MacKie-Mason

On Sat, May 18, 2013 at 11:56 AM, roadiemoose notifications@github.com wrote:

Without ISX or some other tweak(Bulletin etc..) Notification Screen is not visible when the phone is locked.. So removing ISX wouldn't change anything.. The problem is your tweak needs to be able to get permission to call the server to get a location update and then get the weather info when the phone is locked. That's how I see this problem.. Agree?? When the phone is unlocked and I swipe down to show the NC screen, I get a pop up asking to give springboard location permission.. This window obviously can't pop up when the phone is locked, therefore the tweak times out trying to reach the server..

Sorry if I'm repeating myself or saying something you already know..

Reply to this email directly or view it on GitHub: https://github.com/andrewmm/ncwunderground/issues/30#issuecomment-18104354

roadiemoose commented 11 years ago

Update seems to have solved the "update" and location permission issues.. Nice work.. I have some random cosmetic centering issues on my iPad.. It's cure with swiping up to hide NC, then swiping back down to reveal it again.. Not a huge problem at all..

Nice work..

Bryan