-
Notifications
You must be signed in to change notification settings - Fork 16
Major maintenance #355
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
base: master
Are you sure you want to change the base?
Major maintenance #355
Conversation
Couldn't find any problems. |
Thanks Bob! |
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. |
@Mark-H what version of PHP is needed and any other component updates needed? |
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. |
@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… |
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