Document the component preview field for the dynamic zone picker - #3366
Open
pwizla wants to merge 2 commits into
Open
Document the component preview field for the dynamic zone picker#3366pwizla wants to merge 2 commits into
pwizla wants to merge 2 commits into
Conversation
|
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
github-actions
Bot
added
pr: updated content
This was referenced Aug 26, 2026
pwizla
added a commit
that referenced
this pull request
Aug 26, 2026
The previous fix was wrong. "shell: bash" expands to
"bash --noprofile --norc -e -o pipefail {0}", which still carries -e, so the run
kept stopping after 3 of 12 checks. Only the explicit "shell: bash {0}" form
drops it. Reproduced with the real scripts against #3366: -e stops at 3 checks
and exits 1, bash {0} runs all twelve and exits 0.
pwizla
added a commit
that referenced
this pull request
Aug 26, 2026
…3409) The previous fix was wrong. "shell: bash" expands to "bash --noprofile --norc -e -o pipefail {0}", which still carries -e, so the run kept stopping after 3 of 12 checks. Only the explicit "shell: bash {0}" form drops it. Reproduced with the real scripts against #3366: -e stops at 3 checks and exits 1, bash {0} runs all twelve and exits 0.
pwizla
added a commit
that referenced
this pull request
Aug 26, 2026
The host list was written with backslash-escaped dots and passed to awk through -v, where a backslash is a string escape rather than a regex one. awk consumed it, gawk warned "escape sequence \. treated as plain .", and the dot became a wildcard: https://strapiXio.evil.com was accepted as an allowed host. Writing the dots as [.] survives -v untouched. Verified: the four legitimate hosts still pass, strapiXio.evil.com and notstrapi.io are now blocked, and the warning that appeared in the #3366 run summary is gone.
pwizla
added a commit
that referenced
this pull request
Aug 26, 2026
The host list was written with backslash-escaped dots and passed to awk through -v, where a backslash is a string escape rather than a regex one. awk consumed it, gawk warned "escape sequence \. treated as plain .", and the dot became a wildcard: https://strapiXio.evil.com was accepted as an allowed host. Writing the dots as [.] survives -v untouched. Verified: the four legitimate hosts still pass, strapiXio.evil.com and notstrapi.io are now blocked, and the warning that appeared in the #3366 run summary is gone.
pwizla
added a commit
that referenced
this pull request
Aug 26, 2026
All twelve checks ran and the verdict was recorded, but the step still failed. Its exit code is that of its last command, and the last one to run was the test on UNKNOWN: with no errored check that variable is empty, the test returns 1, and the step fails on a perfectly normal outcome. The step reports a verdict, it does not pass or fail on it — eligibility is carried by the output. Reproduced against #3366: same verdict either way, exit 1 before, exit 0 after. The other five steps across both workflows were not affected but get the same explicit ending, so no step's result depends on whichever command happens to be last.
The field is named preview, not screenshot: it was renamed upstream before strapi/strapi#26863 merged. Moved off the Entity Service page, which is deprecated and unlisted, and which the field has nothing to do with.
pwizla
changed the title
pwizla
marked this pull request as ready for review
August 26, 2026 15:02
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 PR documents the optional preview field that components accept in their info object, which displays a thumbnail for the component in the dynamic zone picker instead of the default icon. The field is documented in the model information reference and the Content Manager dynamic zones section. Note that the field is named preview, not screenshot as the upstream pull request title suggests: it was renamed during review before strapi/strapi#26863 merged.
Direct preview link 👉 here