Skip to content

.storeAll now stores processed images for locals too, as it should be#810

Merged
kean merged 1 commit into
kean:mainfrom
HyperfocusDisordered:patch-1
Feb 22, 2026
Merged

.storeAll now stores processed images for locals too, as it should be#810
kean merged 1 commit into
kean:mainfrom
HyperfocusDisordered:patch-1

Conversation

@HyperfocusDisordered

@HyperfocusDisordered HyperfocusDisordered commented Aug 16, 2024

Copy link
Copy Markdown
Contributor

I needed this for correct behavior in my app...
Read closely:

/// Stores both processed images and the original image data.
    ///
    /// - note: If the RESOURCE IS LOCAL (has file:// or data:// scheme),
    /// only the PROCESSED IMAGES ARE STORED**.
    case storeAll

Same for .automatic
/// - note: Store only processed images for local resources (file:// or
/// data:// URL scheme).

Acutally it can be literally ∞ times more flexible if .shouldStoreResponseInDataCache becomes public settable lambda.
Lambda always enables Turing complete flexibility, no matter in which case...

PS i love this lib so much

@kean

kean commented Feb 22, 2026

Copy link
Copy Markdown
Owner

Thank you!

@kean kean merged commit bbd1001 into kean:main Feb 22, 2026
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