Tue, Apr 25
Moving to "Monitoring" on Wikidata Sprint board, as this is not actively work on at the moment.
All patches mentioned in the task description have been either merged or dropped. That means now the new release of DataValues/Geo is needed, and that would be it? Or is there anything else here, which was not mentioned so far?
I've checked the image on my computer. In browsers and inkscape it looks fine. But it in gimp, and ubuntu's "image viewer" it looks broken in a way that Thiemo reported. I guess we should care most of browsers, but still might try to remove some SVGO option, and have a slightly bigger file but with less risk that the icon will be looking wrong.
Mon, Apr 24
Thu, Apr 20
Wed, Apr 19
Sure, I might have a look.
I've submitted https://gerrit.wikimedia.org/r/348939 that expands existing documentation slightly based on your feedback @Lucas_Werkmeister_WMDE. There is also some example config included, which hopefully helps in some places you've spotted as non-obvious.
Thanks @Urbanecm. To have it said explicitly: three RSS/ATOM feeds discussed here have been whitelisted on mediawiki.org in order to use them on WMDE Engineering page on mw.org (see T127176).
https://www.mediawiki.org/wiki/WMDE_Engineering has been recently restructured significantly. Among other changes, those feeds are no longer included there. WMDE has no plans to use any of those feeds on mediawiki.org in the future. As such, those could be removed from the white list to keep it clean and up-to-date.
I've tested the extension locally and on structured-commons Labs instance and I've only discovered T163313. Apart from that the extensions seems to be working OK and doing what it is expected from the beta version.
T163313 is clearly something that should definitely be fixed before the extension is deployed to production but it does not block security review as far as I can tell.
Therefore this task is done.
I can confirm that this feed can be now removed from the white list. https://planet.wikimedia.de/atom.php and https://planet.wikimedia.de/wikidata/atom.php are also not going to be used at mediawiki.org, so they could be removed from the whitelist as well. If you need any other information/confirmation regarding this, please just ping me.
Tue, Apr 18
I haven't digested this proposal, and going even more further as @daniel outlined in his comment at https://gerrit.wikimedia.org/r/#/c/345292/, thus I don't feel ready to comment on the whole proposal.
No need for FAQ at this point has been recognized. Screen cap added. Some more polishing of installation instructions, and minor changes to other section of the page.
With all that, this task is considered done.
Lowering the priority as the original task has been completed a while a ago. This ticket is now reused for the refactoring patch, which is not critical.
Wed, Apr 12
Thank you @Lucas_Werkmeister_WMDE! This is exactly this kind of feedback I wanted to get on this documentation. Your work is very much appreciated!
What @Krinkle has suggested sounds like a good plan at glance!
We just need to make sure that all tests are run in such set up. I will be looking at differences between jobs, and keep spamming here if I find something relevant.
So the basic documentation at mediawiki.org is there. Still no screenshot - suggestion what to include on it are welcome!
Also no FAQ section but I am not sure if there is a need to have right now.
Tue, Apr 11
Uh. Thanks @matmarex for pointing to this! It is that simple then. So Wikibase does the right thing. I've also read the note about the discouraged use of $wgExtensionMessagesFiles fast and wrong. It clearly says ". For setting other variables than $messages, $wgExtensionMessagesFiles should still be used."
Basic installation instructions ready for review at: https://www.mediawiki.org/wiki/Extension:WikibaseMediaInfo#Installation and https://gerrit.wikimedia.org/r/#/c/347366/
I've created some basic documentation, including installation and configuration notes.
The documentation has been put into README.md file (https://gerrit.wikimedia.org/r/347366), and almost identically reproduced at https://www.mediawiki.org/wiki/Extension:WikibaseMediaInfo.
To be clear: this is something which is NOT required to be done before applying the schema change to the database during the data center switch over. It could be implemented later.
Mon, Apr 10
Related patch got merged. Is there something else left to be done in this task?
Fri, Apr 7
Initial version of the feature ready. It would need further improvements. Those should be tracked separately once actual requirements have been specified.
Thu, Apr 6
Wed, Apr 5
Mon, Apr 3
The problem reported in T161698 blocks merging any of Wikibase patches, so this is becoming more pressing for Wikidata folk. Is there anything on the Wikibase side we could do to help fixing the issue?
@aude this is now resolved, right?
@aude this could be closed, no?
I've checked this fixed the issue on Wikibase instances we've installed on Labs. It used to crash over there, now it looks OK. I'd then say, this is fixed.