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

Added layout paragraphs to publication content types instead of existing content fields. #139

Merged
merged 6 commits into from
Feb 6, 2024

Conversation

rupertj
Copy link
Member

@rupertj rupertj commented Jan 29, 2024

Implements #74

@finnlewis
Copy link
Member

Looking at the config dependency errors @rupertj , I think we might need to require layout_paragraphs in composer.json

@rupertj
Copy link
Member Author

rupertj commented Jan 29, 2024

Yeah, you're probably right. Cheers for spotting that, although I wasn't expecting anyone to review this yet :)

@finnlewis
Copy link
Member

Hey @rupertj - sorry, no worries!

I'm just taking advantage of Merge Mondays moving to Merge Tuesdays... so rattle through the list of pull requests at

https://github.com/pulls?q=is%3Aopen+is%3Apr+archived%3Afalse+user%3Alocalgovdrupal+draft%3Afalse

We also can mark pull requests as draft if we want to do more work before review, then they don't show up in that list.

@rupertj
Copy link
Member Author

rupertj commented Jan 30, 2024

@finnlewis Tests are green, and my own manual testing looks good. Could you give this a look? Ta.

@finnlewis finnlewis self-assigned this Feb 5, 2024
@finnlewis
Copy link
Member

Testing now! Sorry not to get to it sooner @rupertj

@finnlewis finnlewis merged commit 41694be into 1.x Feb 6, 2024
8 checks passed
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.

None yet

2 participants