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

Update/gatsby wordpress #612

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Update/gatsby wordpress #612

wants to merge 10 commits into from

Conversation

gilzow
Copy link
Contributor

@gilzow gilzow commented May 27, 2022

Updates template to latest from WordPress + Gatsby and adds automation of WordPress install and dummy content.

adds ids to both app routes to make them easier to locate in deploy scripts
…pendencies)

updates wordpress core to 5.9.X branch
adds wp-gatsby and wp-graphql plugins
adds wp-core-installer and composer installers as allow-plugins
symlinks env.production to deploy/platformsh.env which is built by wordpress during deploy stage
moves and syncs .config that is created during build but would be overwritten since we need .config to be writable later
adds network file share
@gilzow gilzow requested a review from chadwcarlson May 27, 2022 17:51
Copy link
Contributor

@chadwcarlson chadwcarlson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gilzow looks great - want me to merge now or wait until the template is updated?

# Conflicts:
#	templates/gatsby-wordpress/files/.platform/routes.yaml
#	templates/gatsby-wordpress/files/gatsby/package-lock.json
#	templates/gatsby-wordpress/files/wordpress/composer.lock
Updates PR branch to latest from Master
@gilzow
Copy link
Contributor Author

gilzow commented Dec 5, 2022

This should close #612 bringing it and the current WordPress-Composer template into sync

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.

2 participants