Add deferred() conversion option#3922
Merged
freekmurze merged 13 commits intospatie:mainfrom Apr 28, 2026
Merged
Conversation
Member
|
Could you add docs for this? Could you also revert all the imports of the namespaces in the config file. I don't want it to start with a big chunk of imports. |
Contributor
Author
|
@freekmurze I've added some documentation. The style and formatting changes here were made by the |
Contributor
Author
|
I've address an issue with |
Replace the em-dash phrasing with a comma. Add two short paragraphs: one noting that deferred work runs inline in the same PHP process so slow or numerous conversions are better queued, and one stating the Laravel 11.23 minimum for the defer() helper.
Early-return when queueAll is set so the deferred and queued predicates no longer need to thread the queueAll flag.
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
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This adds a
deferred()media conversion option. This is handy if for example you wanted to do a quick conversion such as an avatar thumbnail which is used on the next page after upload.I did think about using a single enum property, rather than toggling two boolean ones, but I didn’t want to break compatibility.
Note: There's a "Fix styling" CI commit that's changed a lot of files here that are unrelated to this PR.