Skip to content

[V14] Fix old rte seed value causing the toolbar not to appear#16665

Merged
Zeegaan merged 1 commit intov14/devfrom
v14/fix/old-rte-seed-value
Jun 28, 2024
Merged

[V14] Fix old rte seed value causing the toolbar not to appear#16665
Zeegaan merged 1 commit intov14/devfrom
v14/fix/old-rte-seed-value

Conversation

@Migaroez
Copy link
Copy Markdown
Contributor

Description

There is a chance that a seeded RTE datatype configuration from the early days of v13 (or before) was never touched and thus its configuration data never upgraded to a newer format. This results in new backoffice not being able to read that configuration and simply not display a toolbar, when editing the configuration all checkboxes for the different tools are unchecked.

This PR introduces a migration that looks for that exact seed value and relaces it with the value that a newly created default RTE datatype would have.

Testing

  • Create a new v13 project, add a document type with the default RTE datatype and add some content for good measure
  • Update the project to v14 => when editing the content, the RTE toolbar should be visible
  • Custom RTE's from v13 with other options should not be affected, i.e. there should be no change in the toolbar buttons available for those editors when upgrading from v13 to v14

…datatypes

Moved 14.1 migrations into their own namespace
@Zeegaan
Copy link
Copy Markdown
Member

Zeegaan commented Jun 28, 2024

Looks good, tests good 🚀

@Zeegaan Zeegaan merged commit d170193 into v14/dev Jun 28, 2024
@Zeegaan Zeegaan deleted the v14/fix/old-rte-seed-value branch June 28, 2024 07:28
Zeegaan pushed a commit that referenced this pull request Jun 28, 2024
…datatypes (#16665)

Moved 14.1 migrations into their own namespace

Co-authored-by: Sven Geusens <sge@umbraco.dk>
(cherry picked from commit d170193)
matthewcare pushed a commit to matthewcare/Umbraco-CMS that referenced this pull request Jun 29, 2024
…datatypes (umbraco#16665)

Moved 14.1 migrations into their own namespace

Co-authored-by: Sven Geusens <sge@umbraco.dk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants