-
Notifications
You must be signed in to change notification settings - Fork 3
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
SMW 2.1 -> 2.5.8 #55
SMW 2.1 -> 2.5.8 #55
Conversation
Just to give a review update, I am definitely still planning on approving -- the thing I want to do before that is independently verify deployment on my GCP dev instance. Thanks for this work! |
7185012
to
a3c4f63
Compare
Argh. Just found this breaks SemanticForms (the "Edit" tab reverts back to the dumb editor, but the extension is still listed in Special:Version). Bisecting it, even just upgrading SMW one version, to 2.2.0, breaks it. |
After waaaay too much debugging at a low level (upgrading, downgrading, printing, dumping, etc), I went back a re-read the docs. https://www.mediawiki.org/wiki/Extension:Page_Forms/The_%22edit_with_form%22_tab A (the?) fix is just to add a line to each Category page (Canyon, Region, etc) defining the default form template. For example: Here: http://ropewiki2.ak/Category:Regions/edit add the line After that, the expected "Edit/Edit source" tabs reappear. It's not clear what's changed in the upgrade (it's definitely tied to moving to above SMW 2.1), but I like this as a fix. Update: This explains it:
As an aside, I don't think we need to keep our own copy of the old SemanticForms repo ourselves. Cloning the PageForms repo and checking out branch 3.7 (the last version of SemanticForms) seems to work fine. |
a3c4f63
to
650d1ef
Compare
I'm testing the default form conversion on some lesser used categories to check they can be switched before the upgrade. e.g. https://ropewiki.com/Category:Knots Update: this is complete for all category pages which had a default form defined. |
1e065f3
to
0a3d310
Compare
0a3d310
to
27ba907
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like you have good testing for the change (thanks for that work!), so let's update :)
Stack created with Sapling. Best reviewed with ReviewStack.