[06:54:37] morning [08:16:17] o/ [08:16:21] o/ joakino [08:26:18] soundtrack of the day: https://soundcloud.com/noisey/rosetta-soot [08:26:39] ^ I'M SEEING THEM LIVE ON SATURDAY I AM SO EXCITED I AM SO EXCITED I AM SO EXCITED [08:26:53] seriously, i feel like i'm a teenager again [08:26:59] i have a crush on this band [08:27:12] :) [09:29:28] bmansurov: thanks for the browserstack loan <3 <3 <3 [09:29:48] maybe we should make it a team account? [09:37:47] e.g. change the email to reading-web-team [09:44:16] phuedx: np, sure [09:44:33] who has access to that email address though? [09:44:35] phuedx: ^ [09:44:46] we're all admins on that list [09:44:49] but good point [09:46:15] phuedx: would you mind sending an email to Sarah R? [09:47:15] not at all [09:47:49] thanks, she created my account, and I'm sure she'll be able to change it to a team account [10:00:57] bmansurov: done, i've cc'd adam just in case [10:01:04] ta [10:01:09] cool [11:01:54] til that doxygen also builds a list of todos in our php source code: https://doc.wikimedia.org/MobileFrontend/master/php/todo.html [13:31:23] * mdholloway -> coffee [14:13:51] mdholloway: i think the aggregated feed cards are actually in the correct order, as they are. [14:15:09] dbrant: cool. i don't have strong feelings about the order one way or another off the top of my head, just thought i'd check [15:36:05] mdholloway: o/ hello! so the app is currently using the summary endpoint for rando articles which i think is correct. howver, prod doesn't seem to have it at the moment. is that expected? [15:36:49] niedzielski: there's a bug in the public random endpoint, mobrovac is deploying a fix as we speak, i think [15:37:21] mdholloway: 👍 [15:37:37] both /page/random/title and /page/random/summary should work after that [15:49:47] mdholloway: {{done}} [15:50:19] mobrovac: \o/ thank you! working like a charm [15:50:51] no no, thank you mdholloway for spotting my stupidity in the last moment :) [15:51:02] s/in/at/ [15:51:27] mobrovac: haha, no worries, stuff happens [15:52:50] looks like we do need an app patch to handle the new response format, though [15:53:53] also, the 'because you read' and 'continue reading' can show non-mainspace pages and probably shouldn't (i just got a File: page, for instance) [15:53:57] patch time [15:54:24] uh [15:54:39] mobrovac: (that last isn't related to the endpoint) [15:54:56] because you read File:bla.jpg, you should also read File:blah.mpeg :) [15:55:41] haha, maybe so :) [16:03:19] testing the random crash fix now, i think it's a one-liner [16:09:07] ok, crash fix is up, gonna grab a bite to eat and bike to coworking [16:37:00] mdholloway|biab: thanks for the fix! [16:38:21] mdholloway: my pleasure! [17:16:41] mdholloway: get up, stand up, stand up for your status [19:55:20] mdholloway: hey there. i accidentally merged dmitry's patch without seeing your comment :| [19:55:43] niedzielski: yeah, no worries, i do wonder why i can't get a different language, though [19:55:44] mdholloway: i tested it out on api 23 by switching it to french and it worked fine for me. can you give it another try? [19:55:52] i know the string resources are there [19:55:56] mdholloway: setting the device language should definitely update the wordmark... [19:56:02] niedzielski: yeah, i'll give it another try [20:01:15] niedzielski: ah, it works [20:01:16] all good [20:01:27] i noticed an interesting RTL bug i'll have to write up, though [20:01:30] phew [20:01:33] thanks :) [20:01:47] np! [22:32:10] dr0ptp4kt - or anyone else familiar with the hovercards instrumentation -, any spontaneous thoughts on https://phabricator.wikimedia.org/T139319#2467334 (improbably low number of disables and settings cog clicks)? [22:32:25] ...or about https://phabricator.wikimedia.org/T140485 ("Schema:Popups logs "dwelledButAbandoned" and link open events at the same time")?