Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use CachedImageProvider and add news & icon list safety checks #2

Closed
wants to merge 1 commit into from

Conversation

jerold
Copy link

@jerold jerold commented Oct 25, 2018

PR addresses issue #1

Replaced the ImageProvider with a CachedImageProvider.

Required updating one other dependency, but analyzer didn't show any issues doing so.

@xsahil03x
Copy link
Owner

Thanks for the contribution

@xsahil03x xsahil03x closed this Jul 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants