Improve/Refactor replaceUUIDsAndUpdateModel() and insertMultipleNodes()

Created on 4 September 2024, 12 days ago

Overview

Follow-up for Implement the concept of sections within the client Fixed .

This is a bit of a code maintenance/tech debt repay job.

Refactor layoutModelSlice.ts::replaceUUIDsAndUpdateModel() and layoutModelSlice.ts::insertMultipleNodes(), make them unit testable, then write unit tests.

If possible also consider if layoutModelSlice.ts::insertMultipleNodes() and layoutModelSlice.ts::insertNode() can be combined in some way into a single reducer that can handle one or multiple nodes.

📌 Task
Status

Active

Component

Page builder

Created by

🇬🇧United Kingdom jessebaker

Live updates comments and jobs are added and updated live.
  • Needs tests

    The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

  • Novice

    It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.

Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024