Skip to content

Major maintenance - #355

Open
Mark-H wants to merge 30 commits into
masterfrom
major-maintenance
Open

Major maintenance#355
Mark-H wants to merge 30 commits into
masterfrom
major-maintenance

Conversation

@Mark-H

@Mark-H Mark-H commented May 1, 2025

Copy link
Copy Markdown
Collaborator

What does it do?

Update Commonmark to v2.6, Slim to v4, and a variety of other dependencies to new major releases.

This is now running on the mirror at https://docs.modx.ORG to help with testing! Pending some infrastructure updates for the deployment to live.

Adds new markdown features: autolinking, footnotes, smart punctuation, easier automatic embeds.

Adds new social images to each documentation page to make sharing them look better.

Why is it needed?

Security, stability, and generally keeping the official documentation site healthy.

Related issue(s)/PR(s)

N/a

@BobRay

BobRay commented May 1, 2025

Copy link
Copy Markdown

Couldn't find any problems.

@Mark-H

Mark-H commented May 2, 2025

Copy link
Copy Markdown
Collaborator Author

Thanks Bob!

@Mark-H

Mark-H commented May 21, 2025

Copy link
Copy Markdown
Collaborator Author

This is ready to deploy as soon as the production server is upgraded to a newer PHP version, which I don't think I have access to do, but I have reached out to Liz about that.

@rthrash

rthrash commented May 28, 2025

Copy link
Copy Markdown

@Mark-H what version of PHP is needed and any other component updates needed?

@Mark-H

Mark-H commented Jun 3, 2025

Copy link
Copy Markdown
Collaborator Author

I think we should first bump production to 7.4, which works for the old and the new versions so we can upgrade the app. But after that we're free to go to 8.3 - that's what the mirror is on at the moment.

@jaygilmore

Copy link
Copy Markdown

@Mark-H in order to "bump" production to 7.4 we need to move it from the server it's on to another machine. I actually started to attempt a migration test yesterday but the platform I moved it to didn't seem to support it. Now. Admittedly I did try to move it directly to 8.3 (the default for the migrations) but I did run composer update. The issue is that I believe that this app requires sqlite and I'm not sure if it's on the target platforms. More soon…

Mark-H and others added 3 commits July 30, 2026 18:56
Implements a DbValueGuard to centralise field length limits and checking. Replaces sqlite native `rowid` columns with an explicit `id` on each schema.
Align Docker/CI/phpcs with 8.3+, and update Symfony Console commands and Monolog for the new major versions.

Co-authored-by: Cursor <cursoragent@cursor.com>
Enable cross-version PHPUnit runs, fix the Slim 4 request harness, and update functional assertions to match current routing.

Co-authored-by: Cursor <cursoragent@cursor.com>
Mark-H and others added 12 commits July 30, 2026 20:47
Serve ads.txt, robots.txt, apple-touch precomposed icon, and Chrome traffic-advice, and short-circuit known bot/scanner paths before fancy 404 handling.

Co-authored-by: Cursor <cursoragent@cursor.com>
Generate per-version/language XML under public/ for ~4k docs URLs, linked from robots.txt and refreshed by sources:update.

Co-authored-by: Cursor <cursoragent@cursor.com>
Keeps Searches and PageNotFound from growing indefinitely via a cron-friendly CLI command, documented in the README.

Co-authored-by: Cursor <cursoragent@cursor.com>
Fixes #122

Co-authored-by: Cursor <cursoragent@cursor.com>
Keeps the OpenCollective sidebar visible regardless. Fixes #46.

Co-authored-by: Cursor <cursoragent@cursor.com>
Convert the template color map to CSS custom properties with light and dark
palettes so surfaces, typography, navigation, search, callouts, tables, and
inline code adapt cleanly. A header button cycles system → light → dark, with
preference persisted in localStorage and a blocking head script to avoid FOUC.
Prism code blocks keep the tomorrow theme for contrast; toolbar and gutter
tokens follow the page theme. Also adds a dark logo variant and forces light
colors when printing.

Co-authored-by: Cursor <cursoragent@cursor.com>
Custom CommonMark renderers read title from the wrong place; use getTitle() so title attributes are emitted. Fixes #311.

Co-authored-by: Cursor <cursoragent@cursor.com>
Fixes #354

Co-authored-by: Cursor <cursoragent@cursor.com>
Root index.md was skipped by the nav walk, so history never got indexed and "Last updated" showed as unavailable on the homepage.

Co-authored-by: Cursor <cursoragent@cursor.com>
Route documentation requests through Doc again so missing markdown can fall through to renderFile for assets in the docs tree.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
No longer accepting raw HTML as documentation sources, offering alternative solutions for local videos (using image syntax)
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.

Updating PHP dependencies: Slim, dotenv, phpunit

4 participants