Skip to content

added static field defaultCacheManager to CachedNetworkImageProvider#933

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

added static field defaultCacheManager to CachedNetworkImageProvider#933
martijn00 merged 3 commits into
Baseflow:developfrom
AurelVU:develop

Conversation

@AurelVU

@AurelVU AurelVU commented Mar 27, 2024

Copy link
Copy Markdown

✨ 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 martijn00 merged commit c5c650f into Baseflow:develop Jul 31, 2024
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