[02:25:33] 06Multimedia, 10Analytics-EventLogging, 10ArchCom-RfC, 06Discovery, and 9 others: RFC: Use YAML instead of JSON for structured on-wiki content - https://phabricator.wikimedia.org/T147158#2683091 (10bd808) The the long time maintainer of [[https://pecl.php.net/package/yaml|YAML bindings for PHP]], I oppose... [07:47:34] 06Multimedia, 10Analytics-EventLogging, 10ArchCom-RfC, 06Discovery, and 9 others: RFC: Use YAML instead of JSON for structured on-wiki content - https://phabricator.wikimedia.org/T147158#2683091 (10TheDJ) >>! In T147158#2683314, @Legoktm wrote: > I think the premise here is flawed. I don't think we should... [07:47:40] 06Multimedia, 10Analytics-EventLogging, 10ArchCom-RfC, 06Discovery, and 9 others: RFC: Use YAML instead of JSON for structured on-wiki content - https://phabricator.wikimedia.org/T147158#2683091 (10jayvdb) If we switched to YAML, we should define a subset that fixes only these problems. Instead I recommen... [07:48:25] 06Multimedia, 10Analytics-EventLogging, 10ArchCom-RfC, 06Discovery, and 9 others: RFC: Use YAML instead of JSON for structured on-wiki content - https://phabricator.wikimedia.org/T147158#2683091 (10RobLa-WMF) I get where Yuri is coming from. YAML's comments and more flexible handling of formatting make it... [07:57:16] 06Multimedia, 10Analytics-EventLogging, 10ArchCom-RfC, 06Discovery, and 9 others: RFC: Use YAML instead of JSON for structured on-wiki content - https://phabricator.wikimedia.org/T147158#2683640 (10Harej) >>! In T147158#2683314, @Legoktm wrote: > I don't think we should ever have users editing raw JSON the... [13:10:18] brion: thedj stood me up on answering questions about https://gerrit.wikimedia.org/r/289976 but I'm getting worried e-mails from people. Think you could take a look? [13:35:03] 06Multimedia, 10MediaWiki-Uploading, 07Community-Wishlist-Survey: Enhance image uploading process - https://phabricator.wikimedia.org/T120745#2684319 (10Aklapper) @Kenrick95: So the underlying issue here is that available license options are too confusing? The sheer number? Missing explanations which license... [14:07:50] 06Multimedia, 10ArchCom-RfC, 10MediaWiki-extensions-CollaborationKit: RFC: JSON validation - https://phabricator.wikimedia.org/T147137#2682376 (10mobrovac) I second the idea of using YAML instead. It's just easier to read and write for humans. The added bonus to this context is that JSON is valid YAML, and c... [14:22:40] (03PS2) 10Matthias Mullie: [WIP] Add 'previous' button throught UW workflow [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/313215 (https://phabricator.wikimedia.org/T122923) [14:26:07] 06Multimedia, 10MediaWiki-Uploading, 07Community-Wishlist-Survey: Enhance image uploading process - https://phabricator.wikimedia.org/T120745#2684422 (10Kenrick95) Summarizing the lengthy description above, I think the problems here are essentially: 1. Missing or incomplete file description 2. No or wrong l... [15:01:53] 06Multimedia, 10Analytics-EventLogging, 10ArchCom-RfC, 06Discovery, and 9 others: RFC: Use YAML instead of JSON for structured on-wiki content - https://phabricator.wikimedia.org/T147158#2683091 (10matmarex) YAML is nice for writing it (you have to worry less about quotes and strings and escape characters)... [15:03:04] 06Multimedia, 10ArchCom-RfC, 10MediaWiki-extensions-CollaborationKit: RFC: JSON validation - https://phabricator.wikimedia.org/T147137#2682376 (10matmarex) We'd still have to validate that YAML in the same way we need to validate JSON (luckily, the formats support the same data structures, so after parsing t... [15:43:15] MatmaRex: Hey, for https://gerrit.wikimedia.org/r/#/c/309767/ we need https://gerrit.wikimedia.org/r/#/c/309766/ merged before the deployment cut tomorrow, right? [15:43:32] Oh, you reverted. Never mind. [15:46:25] 06Multimedia, 10Analytics-EventLogging, 10ArchCom-RfC, 06Discovery, and 9 others: RFC: Use YAML instead of JSON for structured on-wiki content - https://phabricator.wikimedia.org/T147158#2684678 (10bd808) >>! In T147158#2684521, @matmarex wrote: > I honestly don't think introducing YAML content would furth... [16:07:34] marktraceur: sure will poke it over [16:12:45] brion: Great, thanks [16:57:01] 06Multimedia, 06Commons, 10MediaWiki-File-management: If oi_archive_name = '' give error instead of linking to obviously invalid url - https://phabricator.wikimedia.org/T24847#2684962 (10Meno25) [17:19:59] marktraceur: Did you see Ori's e-mail to Ops-l about upload issues (on Friday)? Anything to worry about? [17:20:24] James_F: I saw it, I also saw the reply that said things seemed back to normal, so I assumed it was finte [17:20:25] fine [17:20:39] Kk. [17:55:57] 06Multimedia, 10Analytics-EventLogging, 10ArchCom-RfC, 06Discovery, and 9 others: RFC: Use YAML instead of JSON for structured on-wiki content - https://phabricator.wikimedia.org/T147158#2683091 (10Esanders) JSON with comment stripping seems reasonable. We can still provide pure JSON via a parameter. I don... [18:21:38] (03CR) 10Bartosz Dziewoński: [C: 04-1] "We should be using custom messages for the errors, rather than reusing ones which were meant for different contexts. Otherwise we'll get h" (031 comment) [extensions/FileAnnotations] - 10https://gerrit.wikimedia.org/r/312823 (https://phabricator.wikimedia.org/T145455) (owner: 10Prtksxna) [18:40:57] (03CR) 10Brion VIBBER: [C: 031] "I cannot currently test the ForeignDBViaLBRepo path (for Commons in production) but the local path and the RemoteAPIRepo path (InstantComm" (031 comment) [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/289976 (https://phabricator.wikimedia.org/T61780) (owner: 10TheDJ) [18:41:23] marktraceur: looks good to me, do you have any questions I can help answer or shall I +2 it and we'll confirm on beta? [18:46:28] brion: I have faith in you [18:47:59] :D [18:48:30] (03CR) 10Brion VIBBER: [C: 032] Rewrite discovery of TimedText tracks [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/289976 (https://phabricator.wikimedia.org/T61780) (owner: 10TheDJ) [18:55:35] (03Merged) 10jenkins-bot: Rewrite discovery of TimedText tracks [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/289976 (https://phabricator.wikimedia.org/T61780) (owner: 10TheDJ) [18:56:24] woo [18:59:55] (03PS24) 10Brion VIBBER: Add a new beta feature to enable Videojs [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [19:36:04] (03CR) 10Brion VIBBER: "quick inline notes -- definitely update the messages, and _might_ want to make sure pulling user objects from proper context." (033 comments) [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [19:38:13] (03CR) 10Brion VIBBER: "Note also there's more bugs I want fixed before we actually enable this as a beta feature. :) But I hope to knock out a few fixes in the n" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [19:45:02] (03PS25) 10Paladox: Add a new beta feature to enable Videojs [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) [19:45:18] (03CR) 10Paladox: Add a new beta feature to enable Videojs (032 comments) [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [19:50:11] (03CR) 10Paladox: Add a new beta feature to enable Videojs (031 comment) [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [21:11:16] (03PS3) 10MarkTraceur: Add parsererror message and debug line [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/312532 (https://phabricator.wikimedia.org/T146473) [21:11:36] (03CR) 10Bartosz Dziewoński: [C: 032] "Elaborated the error message a bit to try to absolve us from responsibility for these errors :D" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/312532 (https://phabricator.wikimedia.org/T146473) (owner: 10MarkTraceur) [21:18:07] (03Merged) 10jenkins-bot: Add parsererror message and debug line [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/312532 (https://phabricator.wikimedia.org/T146473) (owner: 10MarkTraceur) [21:18:39] 06Multimedia, 06Commons, 10UploadWizard: 'Unknown error: "parsererror".' while uploading in Upload Wizard - https://phabricator.wikimedia.org/T146473#2686100 (10matmarex) 05Open>03Resolved a:03MarkTraceur New improved error message: "The server responded with an invalid JSON document. This could be a p... [21:51:54] (03PS2) 10Bartosz Dziewoński: Allow cancelling an upload on the "Details" step [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/313214 (https://phabricator.wikimedia.org/T141907) (owner: 10Matthias Mullie) [21:55:02] (03CR) 10Bartosz Dziewoński: [C: 031] "Two changes:" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/313214 (https://phabricator.wikimedia.org/T141907) (owner: 10Matthias Mullie) [21:56:35] (03CR) 10Bartosz Dziewoński: "Also, should we have some confirmation step before removing the upload? Right now it's fairly easy to accidentally lose your input. Maybe " [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/313214 (https://phabricator.wikimedia.org/T141907) (owner: 10Matthias Mullie)