Skip to content
This repository was archived by the owner on May 19, 2023. It is now read-only.

"Stuck" Wallpapers #5

Open
2 of 4 tasks
berwyn opened this issue Aug 23, 2016 · 5 comments
Open
2 of 4 tasks

"Stuck" Wallpapers #5

berwyn opened this issue Aug 23, 2016 · 5 comments
Assignees
Labels
Milestone

Comments

@berwyn
Copy link
Owner

berwyn commented Aug 23, 2016

Currently, the artwork can get stuck on a particular wallpaper due to the next update being scheduled in the past. Additionally, on Nougat (maybe Marshmallow?) the service frequently has no network access (even though it does?)

08-23 10:48:41.374  3392  4345 W MuzeiArtSource: Error fetching, scheduling retry, id=PonyArtService
08-23 10:49:37.305  3392  4369 W MuzeiArtSource: Error fetching, scheduling retry, id=PonyArtService
08-23 10:49:37.310  3392  4369 W MuzeiArtSource: Refusing to schedule next artwork in the past, id=PonyArtService

Similarly, hook up Timber and a logging service to figure out what the connection issue is.

  • Add Timber logging
  • Add some distributed logging host (Crashalytics?)
  • Fix art scheduling issue
  • Implement better error handling scheme
@berwyn berwyn added the bug label Aug 23, 2016
@berwyn berwyn added this to the 1.2 milestone Aug 23, 2016
@berwyn berwyn self-assigned this Aug 23, 2016
berwyn added a commit that referenced this issue Aug 24, 2016
- Use a long to properly store milli values
- Add Timber for basic logging needs
@berwyn
Copy link
Owner Author

berwyn commented Aug 24, 2016

This is partially blocked by muzei/muzei#207 -- Doze causes the service to not have network access, which leads to failed updates. Not much to be done on our end except warn users :/

@berwyn berwyn changed the title "Stuck" Wallpapers caused by art update being scheduled in the past "Stuck" Wallpapers Aug 24, 2016
@coggy9
Copy link

coggy9 commented Jul 11, 2018

The new Muzei 3 API could solve this issue. Will this project be updated to support the upcoming API rewrite coming with Muzei 3?

@berwyn
Copy link
Owner Author

berwyn commented Jul 11, 2018

Well, that API is infinitely better. Yeah, I'm definitely going to update for the new API as soon as I've got time

@coggy9
Copy link

coggy9 commented Oct 15, 2018

Just a FYI if you haven't seen it, but once Google Play enforces the minimum API restrictions for existing apps next month, old API providers will no longer work with the following Muzei update.

@berwyn
Copy link
Owner Author

berwyn commented Oct 18, 2018

I hadn't seen that, thanks for the heads up! Unfortunately, I've been bogged down in large project after large project at work and haven't had a chance to get around to this yet. I guess I'll have to really sit down and focus on this soon :/

@berwyn berwyn modified the milestones: 1.2, 2.0 Oct 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants