Feature: add changes from 1and1 branch, rebase + resolve conflicts for current main#42
Feature: add changes from 1and1 branch, rebase + resolve conflicts for current main#42
Conversation
With this change the tailwind styles are only applied to the app container of the decoupled content store
This stops having to go through all individual worker outputs
Each document needs to be checked whether it’s still connected to the page tree though.
Only upload script once and reuse script and redis instance
BUGFIX: Add html escaping to stderr and stdout of job logs
…ack/Flowpack.DecoupledContentStore into task/adjustments-for-1and1
Previously if `ContentReleaseBatchResult` contained a wrong value or the fallback null the view would crash.
…ce-and-highlight-error BUGFIX: Joblogs preserve whitespace and highlight errors
If a line contains json output it is rendered as toggable details element
skurfuerst
left a comment
skurfuerst
left a comment
There was a problem hiding this comment.
Hey everybody, specifically @Sebobo and @erickloss ,
thanks for your contributions ❤️ Sorry I only now saw this pull request, please ping me next time in Slack :)
I am currently working on a bigger overhaul of the Decoupled ContentStore package, which will support other rendering targets than the default Fusion rendering as well. In the course of doing so, I have reviewed this change, and would like to discuss a few details. Overally, the code looks great ❤️ ❤️ ❤️
I'd love to make this a part of version 2 (my changes will be a bit breaking config-wise, but easy to adjust).
All the best,
Sebastian
PS: I am on holidays the next two weeks, so we'll find a spot for discussing this afterwards :)
| protected $contentReleaseManager; | ||
|
|
||
| /** | ||
| * @Flow\InjectConfiguration("startIncrementalReleaseOnWorkspacePublish") |
There was a problem hiding this comment.
new feature (for changelog / upgrade notes)
…ombinations BUGFIX: use allAllowedCombinations for contentDimensions in document …
fix: Fix errors after resolving merge conflicts for beta v2 release
6 participants
No description provided.