Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sous layout builder recipe #10

Open
wants to merge 55 commits into
base: main
Choose a base branch
from
Open

Conversation

codechefmarc
Copy link

@codechefmarc codechefmarc commented Mar 7, 2024

This PR adds a recipe for layout builder for Sous. To test:

  • Pull this repo down locally
  • Checkout this branch sous-layout-builder-recipe
  • Run lando start
  • Run composer install
  • Run composer run-script post-create-project-cmd
  • (At this point, I've always run into some issues where I would get an error that was discussed previously ERROR Unable to pull down https://github.com/emulsify-ds/emulsify-drupal.git: Error: Command failed: npm install. To fix, immediately cancel out of the script and manually run lando emulsify init sousdrupalrecipe --platform drupal until it runs without error. I have no idea why the timeout exists. Once this is done, manually run every line after this one in the setup.sh script. That seems to do it for me)
  • Also, once it installs without errors, go into the theme folder and run emulsify system install compound and then npm-install and npm run build
  • Then lando drush cr
  • Verify that you can load the site and the theme is installed correctly
  • Run lando install-recipe fourkitchens/sous_layout_builder and verify that the recipe installs correctly
  • Go to the site, add a page, and save.
  • On the view page, click the "Layout" tab to enter layout builder
  • Add a block and verify that the modal comes up and shows 3 blocks: Text with media, text, and video.

Screenshot 2024-03-06 at 5 21 29 PM

  • Verify that you can add these blocks to the page and while adding, the correct fields are available. For example, text with media should have the text field, the media field, and an alignment field.

Screenshot 2024-03-06 at 5 22 44 PM

@laura-johnson
Copy link
Collaborator

Hey Marc,

I needed to add a couple of dependencies to the recipe to get it to install and made a couple of other suggestions here

The thing that doesn't seem right is I can't see the layout tab on the pages.

Suggestions for sous_layout_builder
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants