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

DOCS Clearer sysadmin guidance for "packaging" #9960

Merged

Conversation

chillu
Copy link
Member

@chillu chillu commented Jun 1, 2021

We have all kinds of fun fallbacks that attempt to create supporting files in production environments.
The latest point of contention is dev/build automatically creating files in .graphql/ and public/_graphql/
if those don't exist. That should be regarded as a last resort option to allow introduction of GraphQL v4 in the CMS 4.x release line.

At least since CMS 4.1, we need some form of "packaging" for generated files (public/_resources),
or committing these into the codebase, so let's call that out for anyone running CMS infra.

I've also added a few more cross links between related topics like manifests and caches

We have all kinds of fun fallbacks that attempt to create supporting files in production environments.
The latest point of contention is dev/build automatically creating files in .graphql/ and public/_graphql/
if those don't exist. That should be regarded as a last resort option to allow introduction of GraphQL v4 in the CMS 4.x release line.
At least since CMS 4.1, we need some form of "packaging" for generated files (public/_resources),
or committing these into the codebase, so let's call that out for anyone running CMS infra.
@chillu chillu requested a review from unclecheese June 1, 2021 22:10
@unclecheese unclecheese merged commit fa3c5e6 into silverstripe:4 Jun 1, 2021
@chillu chillu deleted the pulls/4/docs-graphql-and-caching branch June 2, 2021 05:43
@chillu
Copy link
Member Author

chillu commented Jun 2, 2021

Thanks for the review @michalkleiner and @unclecheese! 🤘

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.

3 participants