FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

added static field defaultCacheManager to CachedNetworkImageProvider by AurelVU · Pull Request #933 · Baseflow/flutter_cached_network_image · GitHub

Repository navigation

added static field defaultCacheManager to CachedNetworkImageProvider - #933

Merged
martijn00 merged 3 commits into
Baseflow:developfrom
AurelVU:develop
Jul 31, 2024
Merged

martijn00 merged 3 commits into
Baseflow:developfrom
AurelVU:develop

Conversation

AurelVU commented Mar 27, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor

✨ This pull request is aimed at creating the ability to globally configure the default caching manager

⤵️ Currently, if you need to globally override the caching manager, you need to register it in the constructor of each widget. This is extremely inconvenient if the project grows, and also creates risks that the developer may forget to specify it

🆕 The CachedNetworkImageProvider class now has a static field that can be defined before the application starts. It is the value of this field that will be used in the case when the user did not pass any value to the caching manager

💥 This PR does not make global changes

🐛 It is recommended to check the work associated with switching the default cache manager while the application is running

🤔 Pre-shipment checklist

  • [ +] All build projects
  • [ +] Follows style guidelines (code style guide)
  • Relevant documentation has been updated.
  • [+ ] Reworked based on current development

martijn00 merged commit c5c650f into Baseflow:develop Jul 31, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL