- Issue created by @tedbow
- πΊπΈUnited States tedbow Ithaca, NY, USA
keeping assigned to myself. I am actively working on this
- πΊπΈUnited States tedbow Ithaca, NY, USA
The changes under
/src
were already reviewed by @wim leers in π Provide a utility method to convert client layout and model to server tree and props Active . the the new function\Drupal\experience_builder\Controller\ClientServerConversionTrait::convertClientToServer
is mostly just extracted from\Drupal\experience_builder\ClientDataToEntityConverter::convert
.I added a test for the new function
\Drupal\experience_builder\Controller\ClientServerConversionTrait::convertClientToServer
but this also covered indirectly because it is usedClientDataToEntityConverter::convert
which is gets invoked inApiPublishAllController
andApiContentUpdateForDemoController
so it is covered in their tests. - πΊπΈUnited States tedbow Ithaca, NY, USA
RTBC'ing based of @f.mazeikis approval
-
tedbow β
committed a59d27b3 on 0.x
Issue #3492312: Provide a utility method to convert client layout and...
-
tedbow β
committed a59d27b3 on 0.x
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Nice to see a subset land first! ππ