The iOS team has modified wikidiff2 to allow for returning JSON structured data. Discussed in T228794
Acceptance Criteria:
The iOS team has modified wikidiff2 to allow for returning JSON structured data. Discussed in T228794
Acceptance Criteria:
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Invalid | None | T229661 Core REST API in MediaWiki | |||
Resolved | • eprodromou | T231338 Page history API | |||
Resolved | • eprodromou | T231347 Curator compares revisions | |||
Resolved | tstarling | T231580 Implement GET Revision Comparison | |||
Resolved | • Pchelolo | T231586 Deploy Updated wikidiff2 C++ Engine | |||
Resolved | jijiki | T234175 Deploy wikidiff2 v1.9.0 |
@tstarling @Tsevener has uploaded a preliminary patch at T232231 to provide a JSON structure from WikiDiff2 that they would like input on, could take a look at it?
@WMDE-Fisch @thiemowmde can either of you address this point? I'd like to learn the deployment workflow as well.
Deploy wikidiff2 server side. Note: Need a MW dev to comment on the complexity and substeps of this step
@jijiki Is this unblocked now? I think I see 1.9.0. Let me know if I can help to unblock.
@WMDE-Fisch @thiemowmde @Pchelolo I have installed php-wikidiff2_1.9.0-2~wmf1_amd64 on beta (deployment-mediawiki-07 && deployment-mediawiki-09). If it works as expected, please let me know so to roll it out in production.
@jijiki Thank you! However, I will ask you for one more thing in beta - could you also install it on deployment-mediawiki-parsoid10.deployment-prep.eqiad.wmflabs before proceeding to prod? That's where all the requests to rest.php are routed now, so we need it there to test from outside the beta cluster.
Thank you @jijiki. I was a bit confused in the beginning as it didn't work, but sudo systemctl restart php7.2-fpm.service make it work :)
I've poked around some diffs on beta a bit and it seems to work, I think we can carefully proceed to production.
:( mybad, sorry :(
I will roll out to production tomorrow. The process is to upload the package to our wikimedia-stretch repo, update all servers, and then rolling restart php-fpm.
Mentioned in SAL (#wikimedia-operations) [2019-10-18T10:49:11Z] <effie> Uploading wikidiff2_1.9.0-2~wmf1 to wikimedia-stretch - T231586
@Pchelolo @WMDE-Fisch @thiemowmde version 1.9.0-2~wmf1 is live, please ping me if we need to rollback
I haven't seen sectionTitles returned in the labs compare endpoint output. Even if we aren't passing in section byte offsets yet, I would expect an empty sectionTitles array. This makes me think 1.9.0 doesn't include my latest changes to output section titles for each diff line: https://gerrit.wikimedia.org/r/#/c/mediawiki/php/wikidiff2/+/539906/
So a heads up this may need a version increment and redeployment before section byte offsets can be passed in.
The 1.9.0. release was done before that patch got merged. So I can confirm, that this patch is not part of 1.9.0. - Before we make another release. Is there anything else that should go in and is not merged yet? ^^'
@WMDE-Fisch good point - there are no current outstanding PRs but I can think of a nice-to-have PR that might be good to make and get in now before we go through releasing again. We are also doing user testing next week with some hardcoded data, so might be good to wait and see if anything comes up from that. I'll let you know if we decide to keep it as-is or tweak a bit more for v1. Thanks!
@WMDE-Fisch @eprodromou FYI I have created one more patch for this, then it can be versioned once more and pushed. We do not expect any further changes from user testing next week. Thanks!