Skip to content

Major maintenance#355

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

Major maintenance#355
Mark-H wants to merge 15 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…

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.

4 participants