TAN-7916: Use the Content Builder for all project and folder descriptions#14065
Open
EdwinKato wants to merge 119 commits into
Open
TAN-7916: Use the Content Builder for all project and folder descriptions#14065EdwinKato wants to merge 119 commits into
EdwinKato wants to merge 119 commits into
Conversation
Collaborator
|
…ription migration to reuse
…in a 2-column layout
…M in the folder page e2e test
…ion sidebar renders
…ipation sidebar renders
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
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.
with Claude's help
Use the Content Builder for all project and folder descriptions
All project and folder descriptions now live on the Content Builder — there is a single description format and a single editing surface, with no WYSIWYG (classic) editor.
Three mechanisms combine so that no description is ever off the Content Builder:
RichTextMultiloc, preserving text, images, videos and CTA buttons); blank descriptions are given a default layout.description_multilocis retained, untouched, as a recoverable safety net (it is sunset later in a separate release).Deployment
This PR ships code only — the migration is a no-op until the rake task is run. The task processes all (creation-finalized) tenants in a single run. Dry-run first and capture the output with
tee.HOST=<host>scopes the run to a single tenant (handy for a spot-check before the full run); omit it to process all tenants at once.description_multilocis never modified, and before a disabled layout that held builder content is overwritten its previouscraftjs_jsonis written to the log, so any overwrite is recoverable. The task is idempotent — re-running skips buildables already on the builder.Changelog
Changed
Technical
ContentBuilder::DescriptionLayoutService).ErrorReporter.For translators
app.containers.AdminPage.ProjectDescription.toggleLabelandapp.containers.AdminPage.ProjectDescription.toggleTooltip(the description-builder toggle they labelled has been removed). extract-intl will drop them from the locale files.