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

Quicksilver on Pantheon Doc Update #9223

Open
v0jpantheon opened this issue Sep 12, 2024 · 1 comment
Open

Quicksilver on Pantheon Doc Update #9223

v0jpantheon opened this issue Sep 12, 2024 · 1 comment
Assignees
Labels
Topic: Automation Topic: Code Structure Related to Upstreams and other code structure details Type: Fix Content Issue or PR to resolve incorrect information in the docs

Comments

@v0jpantheon
Copy link
Contributor

v0jpantheon commented Sep 12, 2024

Re: Quicksilver on Pantheon

Priority: Low/Medium/High (choose one, remove the other options)

Issue Description:

The following line can cause confusion especially with folks using Normal Wordpress (non-composer based), which can
prevent them from updating from upstream due to mismatch in directory.

We recommend that you set the web_docroot to true and that you create a dedicated directory under the docroot (for example, web/private/scripts). This tracks files by instructing Quicksilver to look for the files inside the web folder. If your site uses this nested docroot setting, the scripts directory must be located in the web subdirectory of your site's code repository (for example, web/private/scripts).

https://docs.pantheon.io/nested-docroot
Screenshot 2024-09-13 at 1 35 52 AM

Suggested Resolution

Avoid recommending to enable web_docroot since it only is applicable for certain cases.

@rachelwhitton rachelwhitton added Type: Fix Content Issue or PR to resolve incorrect information in the docs Topic: Code Structure Related to Upstreams and other code structure details Topic: Automation labels Sep 25, 2024
@stevector stevector self-assigned this Oct 2, 2024
@stevector
Copy link
Contributor

Yes, this is confusing. Thanks @v0jpantheon! I'll rewrite.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Topic: Automation Topic: Code Structure Related to Upstreams and other code structure details Type: Fix Content Issue or PR to resolve incorrect information in the docs
Projects
None yet
Development

No branches or pull requests

3 participants