Skip to content

feat(ARC-3948): allow images in text 2 columns block - #484

Merged
bertyhell merged 14 commits into
mainfrom
feature/ARC-3948-images-in-text-2-columns
Sep 22, 2026
Merged

bertyhell merged 14 commits into
mainfrom
feature/ARC-3948-images-in-text-2-columns

Conversation

@bertyhell

@bertyhell bertyhell commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

@bertyhell
bertyhell requested a review from reunefe September 21, 2026 15:55
@bertyhell
bertyhell requested a review from reunefe September 21, 2026 17:22

@reunefe reunefe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ik vind de opdeling van deze Content blocken verre van ideaal. Je bent nu deels logica aan het dupliceren. Daarnaast zit je echt wel met een reële impact op bugs zo kort voor een productie release. Ik zou de logica niet uit elkaar trekken en de components laten hoe ze opgebouwd waren

…keiler

fix(ARC-3985): avoid hetarchief translations in avo
@bertyhell

Copy link
Copy Markdown
Contributor Author

Ik vind de opdeling van deze Content blocken verre van ideaal. Je bent nu deels logica aan het dupliceren. Daarnaast zit je echt wel met een reële impact op bugs zo kort voor een productie release. Ik zou de logica niet uit elkaar trekken en de components laten hoe ze opgebouwd waren

maar dan hebben we momenteel een textblock die image logica bevat voor een textblock die gaan image kan bevatten.
Die blokken moeten uit elkaar. Anders is dat echt vuil.
Als dat nog niet stabiel is, dan stellen we de PRD deploy wel uit. meemoo kiest zelf om tegen ons advies in te gaan en toch een complexe wijziging te doen kort voor een PRD release.

@bertyhell
bertyhell requested a review from reunefe September 22, 2026 07:12
@bertyhell
bertyhell merged commit 9985a0d into main Sep 22, 2026
@bertyhell
bertyhell deleted the feature/ARC-3948-images-in-text-2-columns branch September 22, 2026 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants