feat!: universal head injection, drop activeHead
#231
Merged
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.
π Linked issue
β Type of change
π Description
This change drops the use of the implicit shared context provided by the
activeHead
export. Now we explicitly have the user opt-in to a shared head instance or they need to provide their own function to resolve the context.getActiveHead()
, instead you should useinjectHead()
.If you're not processing requests in parallel (such as a testing environment) then doing something like this is safe
Otherwise, you will need to attach the head to a context and use that. For example, in Nuxt we do the below. The
useNuxtApp()
context is provided with unctx.Note: It's important not to pass in a reference to a context within this function unless you're share it will be unique accross requests.
π Checklist