Conversation
Migration Safety CheckFound 4 potential issues: 20260227_230151_add_text_wrap_to_image_text_block.ts Warning (line 6): ALTER keyword detected - review for data loss sql`ALTER TABLE \`home_pages_blocks_image_text\` ADD \`text_wrap\` integer DEFAULT false;`,Warning (line 9): ALTER keyword detected - review for data loss sql`ALTER TABLE \`_home_pages_v_blocks_image_text\` ADD \`text_wrap\` integer DEFAULT false;`,Warning (line 12): ALTER keyword detected - review for data loss sql`ALTER TABLE \`pages_blocks_image_text\` ADD \`text_wrap\` integer DEFAULT false;`,Warning (line 15): ALTER keyword detected - review for data loss sql`ALTER TABLE \`_pages_v_blocks_image_text\` ADD \`text_wrap\` integer DEFAULT false;`,Review these patterns and add backup/restore logic if needed. See |
|
Preview deployment: https://add-img-text-to-post.preview.avy-fx.org |
busbyk
left a comment
There was a problem hiding this comment.
Noticed there was no option to add to Sierra's daily flow
Can you explain this use case a little more?
If the goal is just to have the ability for content editors to include images where the text wraps around them, I wonder if it makes more sense to allow this as an option when embedding MediaBlocks in the ContentBlock/lexical editors rather than expecting them to know they need to use the ImageText component.
The code looks good - just a UX question.

Description
Adds the ImageText block to posts (via the Lexical rich text editor)
Related Issues
Noticed there was no option to add to Sierra's daily flow
Key Changes
BackgroundColorWrappercomponentHow to test
Screenshots / Demo video
Future enhancements / Questions