docs(installation): archive Drupal 7 guide, update installation docs for Drupal 11+ by Copilot · Pull Request #592 · PECE-project/drupal-pece

Copilot · 2026-04-17T23:21:27Z

Created installation-legacy/index.html with D7 archive content
Updated installation/index.html with D11+ content
Fixed installation-legacy/index.html asset and nav paths: changed all ../../ to ../ (favicon, CSS, JS, search, nav links, sidebar links) and base_url from "../.." to ".." — installation-legacy/ is a top-level section like installation/, not a subdirectory
Fixed installation/index.html Nginx version: removed hard-coded 1.1+ and deferred to the linked Drupal system requirements page instead
Added document root note: installation/index.html now explicitly states to configure the web server document root to .../YOUR_PROJECT_NAME/web (not the project root) to avoid exposing vendor/ and composer.json

…Drupal 7 content Agent-Logs-Url: https://github.com/PECE-project/drupal-pece/sessions/4a0859e9-567d-404f-9817-f9fcb0084486 Co-authored-by: revagomes <201890+revagomes@users.noreply.github.com>

… Drupal 11+ Agent-Logs-Url: https://github.com/PECE-project/drupal-pece/sessions/2226b543-d922-4707-bd1d-17140e67cd22 Co-authored-by: revagomes <201890+revagomes@users.noreply.github.com>

…in drush command Agent-Logs-Url: https://github.com/PECE-project/drupal-pece/sessions/2226b543-d922-4707-bd1d-17140e67cd22 Co-authored-by: revagomes <201890+revagomes@users.noreply.github.com>

Copilot

Pull request overview

Updates PECE’s system administration/installation documentation to reflect the move from Drupal 7 to Drupal 11+, while preserving the old Drupal 7 instructions in an archived “legacy” page for reference.

Changes:

Add an archived legacy installation page for Drupal 7, with a banner pointing to the current guide.
Update the main installation guide to Drupal 11+ requirements and a Composer/Drush-based install/update workflow.
Refresh and remove outdated dependency/troubleshooting sections (e.g., php-mcrypt, pecl-yaml, update.php flow).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File	Description
`installation/index.html`	Modernizes installation/update instructions and requirements for Drupal 11+ and links to the legacy guide.
`installation-legacy/index.html`	Adds an archived copy of the Drupal 7 installation guide with navigation back to the current Drupal 11+ guide.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-04-17T23:37:18Z

+    <link rel="shortcut icon" href="../../img/favicon.ico">
+
+
+    <title>System Admin's Guide (Drupal 7 Legacy) - PECE Documentation</title>
+
+
+    <link rel="stylesheet" href="https://use.fontawesome.com/releases/v5.12.0/css/all.css" integrity="sha384-REHJTs1r2ErKBuJB0fCK99gCYsVjwxHrSU0N7I1zl9vZbggVJXRMsv/sLlOAGb4M" crossorigin="anonymous">
+    <link rel="stylesheet" href="https://use.fontawesome.com/releases/v5.12.0/css/v4-shims.css" integrity="sha384-AL44/7DEVqkvY9j8IjGLGZgFmHAjuHa+2RIWKxDliMNIfSs9g14/BRpYwHrWQgz6" crossorigin="anonymous">
+    <link rel="stylesheet" href="//cdn.jsdelivr.net/npm/hack-font@3.3.0/build/web/hack.min.css" integrity="sha384-xJimnBJEGitT5KhSxUYmJBbk+brsHZwt1WFdCNkrNbhVsn5maNIIq2mNUHgZp447" crossorigin="anonymous">
+    <link href='//rsms.me/inter/inter.css' rel='stylesheet' type='text/css'>
+    <link href='//fonts.googleapis.com/css?family=Open+Sans:300italic,400italic,700italic,400,300,600,700&subset=latin-ext,latin' rel='stylesheet' type='text/css'>
+    <link href="../../css/bootstrap-custom.min.css" rel="stylesheet">
+    <link href="../../css/base.min.css" rel="stylesheet">
+    <link href="../../css/cinder.min.css" rel="stylesheet">


installation-legacy/index.html is at the same directory depth as other top-level sections (e.g., installation/, siteadmin/), but its asset URLs use ../../... (favicon/CSS). When the docs are hosted under a subpath (e.g., GitHub Pages), ../../ can escape the site base and break asset loading. These paths should match the rest of the site (typically ../img/..., ../css/..., etc.).

Copilot · 2026-04-17T23:37:19Z

+    <script src="../../js/bootstrap-3.0.3.min.js"></script>
+
+
+    <script src="//cdn.jsdelivr.net/gh/highlightjs/cdn-release@9.18.0/build/highlight.min.js" integrity="sha384-PMPdQfn2gANjihulr7se6A+NaXEsPnvL+jlRzXuav6swX2l/uRnDVSDV+zZCz8E1" crossorigin="anonymous"></script>
+
+    <script>hljs.initHighlightingOnLoad();</script>
+
+
+    <script>var base_url = "../.."</script>
+
+    <script src="../../js/base.js"></script>
+    <script src="../../search/main.js"></script>


The legacy page sets base_url (and related JS includes) to ../.., which is inconsistent with other top-level sections that use ... If this site is served from a subdirectory, ../.. can resolve outside the docs base and break search/navigation URLs. Align base_url and local script paths with the rest of the top-level pages.

@copilot apply changes based on this feedback

Fixed in b1012de. All ../../ relative paths in installation-legacy/index.html (favicon, CSS, JS, search, nav links, sidebar links, and banner links) have been updated to ../, and base_url changed from "../.." to ".." — matching the other top-level sections. Also addressed the related feedback: removed the hard-coded Nginx 1.1+ version in installation/index.html (defers to the linked Drupal system requirements page), and added an explicit note to set the web server document root to .../YOUR_PROJECT_NAME/web to avoid exposing vendor/ and composer.json.

Copilot · 2026-04-17T23:37:19Z

+<li>SQLite 3.45+</li>
+</ul>
+</li>
+<li><p><strong>Web server</strong>: Apache 2.4+ (with <code>mod_rewrite</code> enabled) or Nginx 1.1+.</p></li>


The stated minimum Nginx version (1.1+) is extremely old for a Drupal 11 stack and is likely a typo or too permissive to be meaningful. Consider either (a) updating it to the actual supported minimum for Drupal 11, or (b) avoiding hard-coding a number here and instead deferring to the linked Drupal system requirements page for web server version constraints.

Suggested change

<li>Web server: Apache 2.4+ (with <code>mod_rewrite</code> enabled) or Nginx 1.1+.</li>

<li>Web server: Apache 2.4+ (with <code>mod_rewrite</code> enabled) or a supported Nginx version; see the official Drupal system requirements page below for current Nginx version constraints.</li>

Copilot · 2026-04-17T23:37:19Z

+<h2 id="ive-installed-pece-but-none-of-my-links-are-working-what-is-going-on">I've installed PECE, but none of my links are working. What is going on?</h2>
+<p>Your web server is not properly configured to support URL routing for Drupal.
+Make sure the <code>rewrite</code> module is enabled and your virtual host is configured
+to allow <code>.htaccess</code> overrides (<code>AllowOverride All</code>). For Apache, Drupal ships


These instructions reference the .htaccess in web/, but they never explicitly state that the web server’s document root should be set to the web/ directory (which is important for Composer-based Drupal projects to avoid exposing vendor/, composer.json, etc.). Please add an explicit note to configure Apache/Nginx to serve from .../YOUR_PROJECT_NAME/web.

Suggested change

to allow <code>.htaccess</code> overrides (<code>AllowOverride All</code>). For Apache, Drupal ships

to allow <code>.htaccess</code> overrides (<code>AllowOverride All</code>). Configure Apache or Nginx

to use <code>.../YOUR_PROJECT_NAME/web</code> as the document root (not the project root), so that

files such as <code>vendor/</code> and <code>composer.json</code> are not exposed. For Apache, Drupal ships

Copilot · 2026-04-17T23:37:20Z

+        <li class="second-level"><a href="../../installation/">&#x2190; Current Guide (Drupal 11+)</a></li>
+
+    </ul>
+</div></div>
+        <div class="col-md-9" role="main">
+<div class="alert" style="background-color:#fff3cd;border:1px solid #ffc107;border-radius:4px;padding:16px 20px;margin-bottom:24px;">
+  <strong>&#x1F4C1; Archived — Drupal 7 Legacy Documentation</strong><br>
+  This page contains the original PECE installation instructions for <strong>Drupal 7</strong>, which is no longer maintained.
+  If you are setting up a new PECE instance, please use the
+  <a href="../../installation/"><strong>current installation guide (Drupal 11+)</strong></a> instead.
+</div>


The sidebar and banner links back to the current guide use ../../installation/. From installation-legacy/, that path can escape the docs base when hosted under a subpath. Use the same relative linking pattern as other top-level pages (likely ../installation/).

@copilot apply changes based on this feedback

This was already fixed in b1012de — the sidebar link (line 159) and banner link (line 168) both now use ../installation/ instead of ../../installation/.

… feedback Agent-Logs-Url: https://github.com/PECE-project/drupal-pece/sessions/767c2977-eb7a-4fa4-8e1d-1fff7c1e9441 Co-authored-by: revagomes <201890+revagomes@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-04-18T16:35:44Z

+
+            <!-- Main title -->
+
+            <a class="navbar-brand" href="../..">PECE Documentation</a>


In this top-level section page, the navbar brand link points to "../..", which goes up two levels and can escape the docs root when hosted under a subpath. It should match other top-level sections (e.g. installation/) and link to ".." instead.

Suggested change

<a class="navbar-brand" href="../..">PECE Documentation</a>

<a class="navbar-brand" href="..">PECE Documentation</a>

Copilot · 2026-04-18T16:35:44Z

+
+
+                    <li >
+                        <a href="../..">Home</a>


The Home link in the navbar uses "../.." (two levels up). From /installation-legacy/ this can resolve outside the documentation base; other sections use "..". Update this to ".." for consistent and correct navigation.

Suggested change

<a href="../..">Home</a>

<a href="..">Home</a>

Copilot · 2026-04-18T16:35:44Z

+management guidelines under the “sustainability” section to learn about
+the recommended minimum specs for your servers.</p>
+<h3 id="downloading-the-distro">Downloading the Distro</h3>
+<p>Our distribution is currently being developed on Github. You can obtain


Use the standard capitalization "GitHub" (capital H) in this sentence for consistent branding across the docs.

Suggested change

Our distribution is currently being developed on Github. You can obtain

Our distribution is currently being developed on GitHub. You can obtain

Copilot · 2026-04-18T16:35:44Z

 <blockquote>
-<p>mysql -u $YOUR_USER -p -e "CREATE DATABASE $YOUR_DB_NAME
-CHARACTER SET utf8 COLLATE utf8_general_ci;"</p>
+<p>drush site:install --db-url=mysql://USER:PASS@localhost/DB_NAME --account-name=admin --account-pass=CHANGEME</p>
 </blockquote>
-<p>Proceed to the URL in which your Drupal will reside, i.e.
-https://worldpece.org. From there, you can install PECE like any other
-Drupal site. Follow the Drupal.org <a href="https://www.drupal.org/documentation/install/"><em>official documentation if you need further help</em></a>. In the
-section <a href="http://pece.readthedocs.io/en/latest/installation.html#troubleshooting"><em>Troubleshooting</em></a> below we describe common issues users have when trying to install PECE without the proper backend dependencies, configurations, and permissions.</p>
-<p>One important note: please, make sure to set the permissions on the
-filesystem properly. We cannot emphasize this enough. The <a href="https://www.drupal.org/node/244924"><em>official Drupal documentation explains</em></a> how
-to do so, if you have questions. (In our experience, we have needed to
-set the file permissions at sites/default/files/artifacts to 770 and
-sites/default/files/private to 770.)</p>
+<p><strong>Important:</strong> Replace <code>CHANGEME</code> with a strong, unique password before running this command. Never use a weak or default password on a publicly accessible site.</p>


The Drush example includes the admin password directly on the command line, which is commonly captured in shell history and process listings. Prefer an approach that avoids embedding secrets in the command (e.g., prompting for the password or reading it from a protected environment variable/secret store) and mention that in the docs.

Copilot AI and others added 3 commits April 17, 2026 23:07

docs(installation): update installation guide to Drupal 11+, archive …

f6e4766

…Drupal 7 content Agent-Logs-Url: https://github.com/PECE-project/drupal-pece/sessions/4a0859e9-567d-404f-9817-f9fcb0084486 Co-authored-by: revagomes <201890+revagomes@users.noreply.github.com>

docs(installation): archive D7 guide and update installation page for…

175ee58

… Drupal 11+ Agent-Logs-Url: https://github.com/PECE-project/drupal-pece/sessions/2226b543-d922-4707-bd1d-17140e67cd22 Co-authored-by: revagomes <201890+revagomes@users.noreply.github.com>

docs(installation): add security note for placeholder admin password …

3bc00e2

…in drush command Agent-Logs-Url: https://github.com/PECE-project/drupal-pece/sessions/2226b543-d922-4707-bd1d-17140e67cd22 Co-authored-by: revagomes <201890+revagomes@users.noreply.github.com>

Copilot AI assigned Copilot and revagomes Apr 17, 2026

Copilot created this pull request from a session on behalf of revagomes April 17, 2026 23:21 View session

revagomes requested a review from Copilot April 17, 2026 23:33

revagomes marked this pull request as ready for review April 17, 2026 23:33

Copilot started reviewing on behalf of revagomes April 17, 2026 23:33 View session

Copilot AI reviewed Apr 17, 2026

View reviewed changes

Copilot started work on behalf of revagomes April 18, 2026 15:41 View session

docs(installation): fix asset paths in legacy page and address review…

b1012de

… feedback Agent-Logs-Url: https://github.com/PECE-project/drupal-pece/sessions/767c2977-eb7a-4fa4-8e1d-1fff7c1e9441 Co-authored-by: revagomes <201890+revagomes@users.noreply.github.com>

Copilot finished work on behalf of revagomes April 18, 2026 15:43

Copilot AI requested a review from revagomes April 18, 2026 15:43

Copilot started work on behalf of revagomes April 18, 2026 16:31 View session

revagomes requested a review from Copilot April 18, 2026 16:32

Copilot started reviewing on behalf of revagomes April 18, 2026 16:32 View session

Copilot finished work on behalf of revagomes April 18, 2026 16:33

Copilot AI reviewed Apr 18, 2026

View reviewed changes

	<li><p><strong>Web server</strong>: Apache 2.4+ (with <code>mod_rewrite</code> enabled) or Nginx 1.1+.</p></li>
	<li><p><strong>Web server</strong>: Apache 2.4+ (with <code>mod_rewrite</code> enabled) or a supported Nginx version; see the official Drupal system requirements page below for current Nginx version constraints.</p></li>

-to allow <code>.htaccess</code> overrides (<code>AllowOverride All</code>). For Apache, Drupal ships
+to allow <code>.htaccess</code> overrides (<code>AllowOverride All</code>). Configure Apache or Nginx
+to use <code>.../YOUR_PROJECT_NAME/web</code> as the document root (not the project root), so that
+files such as <code>vendor/</code> and <code>composer.json</code> are not exposed. For Apache, Drupal ships


		<!-- Main title -->

		<a class="navbar-brand" href="../..">PECE Documentation</a>

	<a class="navbar-brand" href="../..">PECE Documentation</a>
	<a class="navbar-brand" href="..">PECE Documentation</a>

	<p>Our distribution is currently being developed on Github. You can obtain
	<p>Our distribution is currently being developed on GitHub. You can obtain

Conversation

Copilot AI commented Apr 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Apr 17, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 17, 2026

Choose a reason for hiding this comment

Uh oh!

revagomes Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 17, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 17, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 17, 2026

Choose a reason for hiding this comment

Uh oh!

revagomes Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented Apr 17, 2026 •

edited

Loading