[00:06:22] jdlrobson: what was the patchset you wanted me to look at yesterday? [00:17:34] awjr: https://gerrit.wikimedia.org/r/#/c/58459/ [00:17:43] ah good, im in the middle of that already :) [00:21:30] New review: awjrichards; "(7 comments)" [mediawiki/extensions/MobileFrontend] (master) C: -1; - https://gerrit.wikimedia.org/r/58459 [00:26:32] jdlrobson: how come https://gerrit.wikimedia.org/r/#/c/58648/3 has a dependency on https://gerrit.wikimedia.org/r/#/c/58647/2? it would be nice if we didn't create unneeded dependencies as it makes code review/merging more complicated [00:27:17] awjr: dependencies are there for a reason :) [00:27:26] awjr: alway review the first one :) [00:27:37] you can't test nearby without it [00:27:41] jdlrobson: i get that, but i dont see why one is a dependency on the other in this case [00:28:05] well awjr if you can't test it how can you review it properly? [00:28:16] i couldn't develop it properly without it :) [00:28:38] well then that's reasonable :p [00:31:49] New review: awjrichards; "I dont think this is the best approach as what you're expecting to be set for $wgNearbyHost to be ju..." [mediawiki/extensions/MobileFrontend] (master) C: -1; - https://gerrit.wikimedia.org/r/58647 [00:35:42] New review: awjrichards; "(1 comment)" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58647 [00:36:25] New review: Jdlrobson; "We currently use this technique for $wgMFPhotoUploadEndpoint" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58647 [00:36:49] jdlrobson: i cant use ^^ with my setup. [00:37:00] awjr: why not? [00:37:25] because you are assuming the URL patterns are the same between enwiki and the dev environment, which in my case is not true [00:37:49] * YuviPanda has not been able to sleep, has gone and reviewed several more patches since [00:37:51] this is addictive [00:37:57] what that does for me is make the request url look like: en.wikipedia.org/testing/core/api.php [00:38:02] which… obviously doesn't work :p [00:39:23] what are you setting as the host? [00:39:30] en.wikipedia.org [00:39:56] set it to http://en.wikipedia.org/ [00:40:16] I assume you set wgScriptPath ? [00:40:30] the problem is the script path, not the host [00:40:47] look at what you're doing on line 31 of jabascripts/specials/nearby.js [00:41:05] if $wgNearbyHost is not set, you just use the API url (which i believe is script path + api.php) [00:41:23] the API url in my case is /testing/core/api.php, which is not the same as that for en.wikipedia.org [00:41:40] im suggesting that you just use the whole URL for the api endpoint in wgNearbyHost instead of trying to set just the host [00:42:02] like you guys wound up doing with the photo endpoint (i think i had this same convo with juliusz) [00:42:31] lol s/jabascripts/javascripts [00:42:36] huh [00:43:22] awjr: what does mw.config.get( 'wgScriptPath', '' ) + '/api.php'; return for you? [00:43:27] just /api.php ? [00:43:53] mw.config.get('wgScriptPath') + '/api.php'; [00:43:54] "/testing/core/api.php" [00:44:08] so oh i see what your saying [00:44:15] my wgScriptPath is different than that of enwiki [00:44:19] ok yeh i'll make it an endpoint [00:44:21] :) [00:44:23] thanks [00:45:07] i think with that change it will be mergable [00:59:56] [Commons-iOS] montehurd opened pull request #40: More small screen updates (master...moreSmallScreenTweaks) http://git.io/pAZrmw [01:12:42] New patchset: Jdlrobson; "Story 419: Nearby distance enhancements" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58847 [06:06:35] New review: Siebrand; "i18n/L10n LGTM as discussed on IRC." [mediawiki/extensions/MobileFrontend] (master) C: 1; - https://gerrit.wikimedia.org/r/58847 [08:39:41] New patchset: Zfilipin; "Updated Ruby gems" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58863 [11:20:55] New patchset: Zfilipin; "wmf-selenium-bot should not report SCM changes in IRC notifications" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58870 [13:53:43] YuviPanda: yeah that's about what i expected really [13:54:00] maybe ~10% of that first flurry were actually useful [13:54:36] moogsi: yeah [13:54:46] that's why people were annoyed i think - the people who do the frontline maintenance stuff are pretty insular and there's not many of them [13:54:54] + they weren't expecting it = upsetness [13:55:12] moogsi: true. [13:55:36] moogsi: I also think that we were not expecting that many, uh, selfies [13:55:47] haha [13:55:49] and preachy proclamations of the return of the christ [13:56:03] moogsi: the running joke was a flood of penises, not selfies [13:56:26] surprised by the lack of genitals in general, really [13:56:52] dick shots are a special brand of selfie [13:58:19] one thing i know after seeing what comes into Commons is, people are now conditioned to spread pictures of themselves everywhere, indiscriminately and apparently to no end [13:58:23] THANKS FACEBOOK [13:58:37] moogsi: yeah [13:58:37] moogsi: :D [13:58:38] moogsi: that is true. [13:59:22] moogsi: I think we all got blindsided by facebook, tbh [13:59:33] moogsi: if we had done it right, people would be addicted to wikipedia, not facebook [13:59:52] haha wikipedia would have to be very different [14:00:12] true [14:00:45] it's quite difficult to be so superficial and get any meaningful work done at the same time [14:01:13] moogsi: true [14:01:19] moogsi: but that's also guided by the software itself [14:01:41] no it definitely could be easier to talk to people [14:02:04] moogsi: Flow is coming [14:02:07] moogsi: Echo is coming [14:02:09] moogsi: Mobile is coming [14:02:12] ooh [14:02:23] moogsi: 2013-2014 is going to be nice :) [14:02:30] you sound like an action move trailer [14:02:32] :P [14:02:38] moogsi: sooooon! [14:02:38] :P [14:02:42] uh huh [14:02:48] FALL 2018 [14:02:52] or.. whenever [14:02:58] moogsi: one of the other things we've in the background is support for h264 / aac support [14:03:01] FALL, WHEN IT'S DONE [14:03:17] moogsi: that should explode our audio / video stuff [14:03:21] ooh fancy [14:03:40] moogsi: but I think a good number of people on commons will come at us with Pitchforks [14:03:54] proprietary formats... [14:04:06] moogsi: well, they aren't proprietary [14:04:16] moogsi: they're patent encumbered in certain parts of the world [14:04:20] patent-encumbered formats? [14:04:22] ah [14:04:45] moogsi: and we won't be violating laws either [14:05:27] i was talking to some dude about uploading .wavs [14:05:37] moogsi: wav has no patents on it [14:06:07] i don't know what's wrong with it [14:06:09] moogsi: we'll be using free software to do our encoding and decoding too [14:06:12] moogsi: oh, it's just huge [14:06:18] i guess MS owns it kinda? [14:06:21] moogsi: think about why we don't accept BMP for example [14:06:22] moogsi: not really [14:06:31] no, people seems to think there would be some philosophical objection [14:06:42] moogsi: there was, on the bug [14:07:03] because Commons is pathologically dedicated to freedom [14:07:19] hehe [14:07:21] but, we have .GIF i guess [14:08:08] moogsi: those patents expired [14:10:20] moogsi: and we have limited GIFs too [14:10:57] moogsi: https://commons.wikimedia.org/wiki/File:Child_running_through_gravity_illusion_room_at_Children%27s_Museum_of_Pittsburgh.gif [14:10:58] :D [14:12:29] moogsi: there seems to be surprisingly little spam in the Android category :) [14:12:36] I just had to mark one image for deletion! [14:12:39] in one entire page! [14:13:36] moogsi: https://commons.wikimedia.org/wiki/File:Chicken.gif [14:15:44] haha [14:16:21] i saw that one with the kid [14:16:54] moogsi: yeah, that's the kid of ragesoss :) [14:17:03] anyway this is why he wanted .wav: https://www.mediawiki.org/wiki/User:Rahul21/Gsoc [14:17:45] now it seems like mayybbeee they will do the encoding into .oga on the server? [14:18:10] moogsi: also https://commons.wikimedia.org/wiki/File:Snake_can_be_completed.gif [14:18:20] moogsi: that code for wav to flacc already exists [14:18:24] moogsi: just needs to be turned on [14:19:09] more gifs!! [14:19:12] moogsi: the second gif is amazing [14:19:47] takes me back to the 90s where i knew someone who could finish nokia snake every time [14:20:21] :) [14:20:27] the screen was smaller of course [14:20:38] heh [14:21:23] moogsi: only 3 deletions in ~1000 images that I had to mark :) [14:21:29] actually 4 [14:21:31] hmm [14:21:31] 5 [14:21:35] but 3 were test uploads [14:21:41] well the android users already know Commons [14:21:49] true [14:21:50] so good [14:48:50] New review: Cmcmahon; "maintenance" [mediawiki/extensions/MobileFrontend] (master); V: 2 C: 2; - https://gerrit.wikimedia.org/r/58863 [14:48:50] Change merged: Cmcmahon; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58863 [14:49:43] New review: Cmcmahon; "less IRC spam from wmf-selenium-bot" [mediawiki/extensions/MobileFrontend] (master); V: 2 C: 2; - https://gerrit.wikimedia.org/r/58870 [14:49:44] Change merged: Cmcmahon; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58870 [15:22:15] https://bugzilla.wikimedia.org/show_bug.cgi?id=46086 oh Rillke :( [15:22:28] we need that one fixed and he decides to go off on a rant [15:22:47] * YuviPanda looks [15:22:56] it's very woolly [15:23:25] basically none of our delete tools work properly because of some unspecified API error [15:24:34] moogsi: 'none'? [15:25:21] well they all rely on the 2 he links in the thread somehow i think... [15:25:36] if Rillke gets hit by a bus or something then we're totally screwed [15:26:07] unfortunately he is busy doing actualy work at the moment :/ [15:26:08] moogsi: AjaxQuickDelete? [15:27:38] moogsi: people can always take up and fix bugs, I guess [15:27:42] it's just javascript [15:29:34] i am trying to figure out where the problem is.. it apparently started just before i even had admin so i don't know how things are supposed to work [15:29:46] heh [15:29:56] most of the scripts are integrated in a way that you just turn them on in preferences and voilà [15:30:40] right [15:32:13] but deletion seems very slow and fails with that error like 1 in 4 times if you send a lot of requests at once [15:32:49] i should talk to #mediawiki , but really Rillke should do it and he.. doesn't want to right now :l [15:32:59] moogsi: sigh [15:33:08] moogsi: is that the case always? [15:33:15] moogsi: maybe it's hitting a part of the API that's got some bad code [15:33:21] no i think he is really busy with actual work right now [15:33:29] oh i thought you meant rillke :P [15:34:55] i'm not sure, i'd have to find a whole mass of files to delete at once [15:36:30] moogsi: :P [15:36:41] moogsi: heh, maybe the mobile team could help with that :P [15:37:01] https://commons.wikimedia.org/wiki/User_talk:Rillke/Discuss/2013 bottom of the page there is what happened earlier, it's weird [16:36:01] YuviPanda: how have you been liking trello? [16:37:10] awjr: pretty good! [16:37:23] awjr: Labels are working well + filtering [16:37:38] awjr: though they are limited in number. But that might be a good thing. Unsure [16:37:44] we haven't hit that limit yet [16:37:59] YuviPanda: what's the URL for apps again? [16:38:06] moment [16:38:26] awjr: so, all the boards wmf uses are at https://trello.com/wikimediafoundation [16:38:48] awjr: and apps team at https://trello.com/board/mobile-app-team-sprint-6/5162461b43476994540002ba [16:38:58] thanks YuviPanda [16:39:02] :) [16:39:05] i appreciate how lightweight it is [16:39:10] so are you creating a new board for each sprint? [16:39:21] awjr: so, we don't know yet. [16:39:34] awjr: we can either create a new board, or just re-use this one. [16:40:00] awjr: if I re-use this one, I'll just rename current 'Done' to 'Sprint No. 6' and archive that list alone [16:40:03] thus we have a historical record too [16:40:11] but then we don't have a record of what was left in the backlog [16:40:16] ah coo, yeah i was giong to ask about history [16:40:19] and that can be fixed by using a new board [16:40:27] can you move stuff between boards? [16:40:27] we can 'copy' this board, create a new one, and archive the 'Done' [16:40:39] awjr: you can copy baords to create new ones [16:40:49] ah ok but you can't move individual things between boards? [16:40:49] awjr: and archive older boards [16:40:58] awjr: so that'll work for history [16:41:40] YuviPanda: are you guys keeping a general backlog somewhere? [16:41:50] awjr: there's one on the Wiki page [16:44:37] cool - thanks YuviPanda :) [16:44:44] np awjr [16:45:04] awjr: also reminder that I won't be in standup, being friday :) [16:45:09] Sigh. [16:45:11] yup, thanks YuviPanda [16:45:15] MaxSem|android: internet trouble? [16:45:34] awjr: but I'll still be around. Working on UploadWizard today [16:45:45] so poke me if you need anything [16:45:50] cool thanks [16:46:14] Yeah. And this time it looks scarier [16:46:25] what's the problem MaxSem|android? [16:46:44] MaxSem|android: no power either? [16:46:45] or? [16:48:02] Gateway not pingable [16:48:08] Not the overloaded billing server as it was before [16:49:59] MaxSem|android: :( [16:51:38] MaxSem|android: are you giong to be able to join us for standup? [16:53:35] So here's my report: worked on variable resources, worked with Kai on OSM in labs. Today will try to work on GeoData bugs [16:54:07] New patchset: Jdlrobson; "Dev Change: Allow nearby page to run on a different api" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58647 [16:54:56] New review: Jdlrobson; "Juliusz put this in place - not sure what other alternatives there are. In meantime I can enable it ..." [mediawiki/extensions/MobileFrontend] (master) C: -2; - https://gerrit.wikimedia.org/r/58627 [16:55:22] New patchset: Jdlrobson; "Rewrite search as an overlay" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58332 [16:55:26] New patchset: Jdlrobson; "Deprecate M.utils" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58400 [16:57:12] New patchset: Jdlrobson; "Story 430: Promote nearby to beta" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58842 [16:57:13] New patchset: Jdlrobson; "Story 419: Nearby distance enhancements" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58847 [16:57:42] New review: Jdlrobson; "Needs more design input" [mediawiki/extensions/MobileFrontend] (master) C: -2; - https://gerrit.wikimedia.org/r/58825 [16:59:27] wool it's standup time [16:59:30] * brion goes to find glasses [16:59:41] * brion adds woop to spell dictionary :P [17:01:37] * YuviPanda wraps brion in wool [17:01:46] mmmm comfy [17:07:26] thanks MaxSem|android [17:08:14] MaxSem|android: were you able to coordinate in-house testing of wgMFVaryResources on testwiki? [17:08:58] MaxSem|android: i ask in part because im about to cover for tomasz in the weekly deployments meeting and i want to report our deployment plan for that change [17:19:23] mhurd: did we finish this one? https://bugzilla.wikimedia.org/show_bug.cgi?id=47083 [17:19:27] * brion checks pull requests [17:19:39] yeah there it is https://github.com/wikimedia/Commons-iOS/pull/40 [17:19:43] * brion tests [17:20:18] brion: yes i believe so [17:23:53] mhurd: a couple quick notes on https://github.com/montehurd/Commons-iOS/commit/a9d47173690fe6cbeafcaf465ce3b56e9b358335 [17:24:20] ugh i have got to remember how to change tab-complete to be case-insensitive in my shell [17:24:31] that really should be default on mac since the FS is case-insensitive :P [17:24:49] brion: commenting now [17:24:56] whee [17:26:07] brion: added comments below your notes [17:26:16] brion: that missing chunk is weeeird! [17:26:27] yeah it's crazy [17:26:41] one of the reasons i always do a 'git diff' before committing [17:26:52] it seems to regen the section ok so i don't know if it's unnecessary data or what [17:26:54] brion: good practice! [17:27:00] but it's nicer to have clean diffs :) [17:27:05] ya [17:30:19] brion: mhurd I have been using git commit -p all the time now [17:30:23] brion: mhurd no missing chunks! [17:30:29] :) [17:30:38] good idea [17:30:56] mhurd: let's disable the 'hide keyboard' button on iPad, it's not needed in portrait mode and in either orientation there's a hide button already on the keyboard [17:30:59] pesky chunks :) [17:31:01] (which i wish iPhone had too :P :)_ [17:31:24] brion: agh! that's a good point! [17:31:37] will do now [17:31:53] tx :D [17:32:20] darn, i left my 3.5" iPod at the office. i'll have to use the simulator [17:33:46] we should see if we can figure out how to make the description cell expand on demand... [17:33:51] or else move it to the bottom :) [17:33:55] no rush on that though [17:35:06] yes i'll make note [17:35:35] mhurd: i'm thinking we should put a solid background behind the 'hide keyboard' button, it looks a little weird if you scroll down over the license chunk [17:35:43] other than that i'm happy, i'll go ahead and merge this so far :D [17:35:59] brion: just about to push that non-ipad commit [17:36:21] [Commons-iOS] brion pushed 6 new commits to master: http://git.io/jMgSGA [17:36:21] Commons-iOS/master 1bbdb16 Monte Hurd: Choose pic button iPad only worked the first time.... [17:36:21] Commons-iOS/master a9d4717 Monte Hurd: Make picture details screen scroll with keyboard... [17:36:21] Commons-iOS/master c811708 Monte Hurd: Image title and description keyboard dismissal... [17:37:23] brion: pushed it [17:37:50] same branch? [17:37:52] brion: ya [17:37:55] * brion looks [17:38:03] will add background [17:39:30] spiff [17:44:48] oh microsoft [17:44:51] " Congratulations! Wikipedia, release 7 is now in the Windows Store. Use this link to your app’s listing in the Windows Store to let others know about your app." [17:44:57] http://apps.microsoft.com/windows/en-US/app/wikipedia/6b80bf54-1a31-4651-acce-59a5b5c2b7c8 <- says release 6 :P [18:09:52] brion: remember when i played with the title box having that slight background color? *that's* why i added that extra space on the left of the title. now that there's no background color it makes little sense to have the padding space haha [18:10:01] ahhh that was it eh [18:10:11] ok easy to take that back out :D [18:10:49] brion: ya i just looked at it on the device and realized how useless it was now :) [18:14:42] * brion updates parallels, whee [18:17:40] grr microsoft still lies. i see r6 in the store still [18:18:11] * awjr slaps microsoft [18:18:12] add to this that EVERY TIME i log into the windows store developer portal it emails me a new confirmation code [18:18:23] >_< [18:18:33] i just want it to work and not take much of my work time :) [18:19:30] obligatory: did you use MSIE? [18:19:42] i *kid* [18:19:55] hehe [18:20:15] even in the win8 store app it's showing the previous revision. i guess i just have to wait… there's no more buttons to push in the dev console [18:20:31] brion: got the "hide keyboard" background color working. playing with colors/transparency now [18:21:07] sweet [18:21:25] brion: also made the focus shift happen if tap lands *anwhere* in cell rather than just on label. even better! [18:21:33] oooooh nicely done [18:24:12] New review: awjrichards; "you may want to rely on $wgCookiePrefix in the cookie name to make it easier to identify this cookie..." [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58641 [18:31:40] New review: awjrichards; "much better, thanks!" [mediawiki/extensions/MobileFrontend] (master); V: 2 C: 2; - https://gerrit.wikimedia.org/r/58647 [18:31:52] New patchset: awjrichards; "Rewrite nearby as an overlay" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58648 [18:32:15] dr0ptp4kt, ping [18:32:22] yurik, pong [18:32:29] hi! [18:32:52] have you tried the new patch by any chance? [18:33:12] not yet. within the next hour, though. [18:33:20] awesome -- if i put the macbook in the right spot on the ChromeBook Pixel's box, the magnet latch in the box makes the macbook think its own lid is closed and it goes to sleep [18:34:23] yurik: no mountains? [18:34:40] Change merged: jenkins-bot; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58647 [18:34:56] YuviPanda, nah, i just walked to the city (the entire hotel staff thought i was crazy to walk 5 km to the center of the city) [18:36:35] they might be right too [18:36:36] :D [18:38:21] dr0ptp4kt, how are you feeling? [18:38:27] still in the hospital? [18:38:53] YuviPanda, of course they are right! noone in their right mind works with wiki ;) [18:51:42] Change merged: jenkins-bot; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58648 [18:51:57] yurik, i feel like usual except for the darn leg. that reminds me, i need to send a note to the team. will do that right now.yurik, you be safe now. [18:54:08] hehe [18:54:55] dr0ptp4kt, so do you want to jump in on this? One thing i could defiently use already is a littel scrlipt to upload settings: ) [18:56:06] yurik, you referring to an ongoing integration or to the refactor or something else? [18:56:30] the ongoing migration to the new settings (refactoring i guess) [18:58:51] gotta mail a computer, brb [19:01:49] dr0ptp4kt, but i don't want to stress you while you are recovering :) [19:01:58] yurik, sure thing. you want the auto-generation script, an upload script, or both? if both, which one is needed first? my thought on auto-generation is we can run the auto-generator then manually cross-check. [19:02:09] yurik, no stress here! this is work! [19:02:29] dr0ptp4kt, lol, for most ppl work == stress [19:03:32] dr0ptp4kt, not sure what you mean by autogeneration script - we need to get all the data together into one table for easy analysis [19:03:49] and later - to take that one big table and uplaod it as individual pages [19:04:44] okay, so one utility that grabs the current config from the 3 pages, turns it into a consolidated data structure, then provides the ability to export that to one JSON blob per carrier. correct? [19:04:46] New patchset: awjrichards; "Surface page images on nearby" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58652 [19:04:58] yep [19:05:11] oh, why 3? [19:05:17] dr0ptp4kt, its actually two i think [19:05:22] plus the VCL file [19:05:38] plus the i18n php file [19:06:06] it doesn't have to be a nice script - this is a one time change, can use search replace :) [19:06:18] but whichever is most convinient [19:06:45] i'm more worried about the upload - because it might need to be repeated several times until we iron out all kinks [19:09:27] yurik, agreed. [19:11:15] dr0ptp4kt, just keep in mind that this is a one time conversion process, so at a later time the most we will need to do is update all existing setting pages with some value [19:11:38] for example we might decide to add an extra value, or to delete some existing value [19:11:47] adding would actually be more rare [19:12:11] but in short - some minor modification scripted somehow [19:12:48] yurik, got it [19:12:57] :) [19:14:35] LOL [19:31:28] adam, re file - i just realized that it might be good to use CSV instead of tabs because this will allow us to have all IPs for one ID in one cell [19:31:32] just an adea [19:31:41] New review: awjrichards; "Yeah page images! w00t!" [mediawiki/extensions/MobileFrontend] (master); V: 2 C: 2; - https://gerrit.wikimedia.org/r/58652 [19:31:58] New patchset: awjrichards; "Bug 44959: Translate wiki related fixes" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58735 [19:32:00] this way multi-line values are surrounded by the " [19:32:20] and excel (and probably libreoffice) will understand it [19:32:30] just replied to the email [19:33:08] Change merged: jenkins-bot; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58652 [19:34:25] jdlrobson, you should torture max about that geo-math stuff :) [19:34:45] that's his baby i think [19:36:55] Change merged: jenkins-bot; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58735 [19:37:15] New patchset: awjrichards; "Style nearby distance" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58824 [19:39:20] mhurd: how's the changes coming? [19:41:14] brion: oh the button change is done. just tracked down and fixed an odd bug: the keyboard hide notifications in the login view controller were being received even when the login view wasn't visible. noticed a layout weirdness which tipped me off. just finished the fix [19:42:00] brion: going to verify quickly then push these up to that same branch [19:42:03] spiff [19:43:42] grr [19:43:43] grr [19:44:13] MaxSem, 100% agree [19:44:48] * yurik contemplates adding animal sounds to the smilei dictionary [19:48:55] MaxSem: get your internet woes sorted? [19:49:02] yup [19:49:14] anything i missed? [19:49:29] New patchset: Jdlrobson; "Promote addToBodyAttributes to desktop skin" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58257 [19:49:29] New patchset: Jdlrobson; "Cleanup Skin language and variants" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58459 [19:49:29] New patchset: Jdlrobson; "Promote menu button generation to MinervaTemplate" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58990 [19:49:30] New patchset: Jdlrobson; "Define render function in MinervaTemplate to output page" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58991 [19:49:30] New patchset: Jdlrobson; "Kill navigationStart" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58992 [19:49:30] New patchset: Jdlrobson; "Promote all the banners into MinervaTemplate" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58993 [19:49:30] New patchset: Jdlrobson; "Move header to generic MinervaTemplate" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58994 [19:49:31] New patchset: Jdlrobson; "banner fixup" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58995 [19:49:31] New patchset: Jdlrobson; "Code move: Lift and shift html rendering to MinervaTemplate" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58996 [19:49:32] MaxSem, i heard a new tech allows multiplexing internet connection over multiple lines... [19:49:32] New patchset: Jdlrobson; "Make the universe explode by making a desktop AND mobile skin called Minerva" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58997 [19:49:48] nothing particularly exciting, MaxSem [19:50:03] MaxSem: were you able to coordinate some in-house testing of wgMFVaryResources on testwiki? [19:50:30] nope [19:51:29] Change abandoned: Jdlrobson; "See https://gerrit.wikimedia.org/r/#/c/58997/" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/56351 [19:53:13] brion: just sent pull request [19:53:13] [Commons-iOS] montehurd opened pull request #41: Hide keyboard button background color and further fixes. (master...moreSmallScreenTweaks) http://git.io/3xsAkA [19:53:32] [Commons-iOS] brion pushed 5 new commits to master: http://git.io/eDG89w [19:53:32] Commons-iOS/master 5ede431 Monte Hurd: Only show "hide keyboard" on non iPads [19:53:32] Commons-iOS/master d2ba61e Monte Hurd: Focus to title and desc if respective cell tapped... [19:53:32] Commons-iOS/master 2c91775 Monte Hurd: Added background to "hide keyboard" button... [19:53:33] brion: lol that was fast [19:53:41] heh i already merged it manually :D [19:54:03] brion: would have been almost scary fast otherwise haha [19:54:12] it's looking awesome so far dude! [19:54:22] anything else you think we should squeeze in before a beta? [19:54:24] brion: woohoo! [19:54:38] brion: lemme check real quick [19:54:41] k [19:56:15] brion: i think it's cool for now :) [19:56:22] eeeexcellent [19:56:29] i'll bump the version and beta it [19:56:37] and upload to apple and wait. :) [19:56:53] awesome! [19:59:08] [Commons-iOS] brion pushed 1 new commit to master: http://git.io/asJBUw [19:59:08] Commons-iOS/master 75c1dce Brion Vibber: Bump version to 1.0.0 [20:01:25] wtf [20:01:26] "Could not launch “Commons-iOS”" [20:01:30] "failed to get the task for process 5065" [20:02:48] jdlrobson: im not seeing 'last modified' and page images look squished in https://gerrit.wikimedia.org/r/#/c/58824/3 - is that known/intentional? [20:02:49] ah i see the debugger doesn't work when i sign for ad-hoc distribution [20:02:54] * brion slaps ios [20:03:03] awjr: last modified is a separator story [20:03:19] yeah, just wanted to dbl check [20:03:28] about that specific patchset [20:03:46] awjr: what do you mean by pageimages being squashed? it should be exactly the same as watch list [20:04:00] does that patchset effect it? [20:04:50] one sec - lemme compare against watchlist and i'll get some screenshots for ya [20:05:02] k [20:05:16] it might be due to the lack of a modified time - maybe that makes the rows slightly bigger [20:05:35] that patch set was just focusing on the distances [20:05:50] (and i expect vihba when i grab her will have some tweaks to make to it anyhow :) [20:06:22] yeah jdlrobson so on watchlist, the thumbs for me are 70x63 but on nearby they are 70x54 [20:06:31] mm that's odd [20:06:34] want screenshots or is that enough for ya/ [20:06:48] * jdlrobson wonders if the api does something different [20:06:54] awjr: i'll have a quick poke around [20:06:57] k [20:07:49] ah i see. watch list does a bit of additional processing [20:07:56] but awjr that patch set has nothing to do with page images :) [20:08:03] as an aside, im also getting an interesting warning on nearby in chrome desktop: Resource interpreted as Image but transferred with MIME type text/html: "https://m.virtualbot.com/testing/core/index.php/Special:Nearby". [20:08:23] awjr: i'll write a tweak (since https://gerrit.wikimedia.org/r/#/c/58652/ introduced it) [20:08:23] jdlrobson: well it has to do with styling, so i was comparing against the mockup, so i just wanted to make sure [20:08:42] awjr: the styling is a separate story [20:08:47] yes i know jdlrobson [20:08:49] k [20:09:04] just didn't want to block this patch set getting merged on this - it's a problem in master gith? [20:09:40] actually it's different in master [20:10:00] thumbs are 70x68 [20:10:32] so… different from the patchset, and different from watchlist :p [20:10:53] huhz!?! how.. [20:11:24] awjr: sure you are not caching? [20:11:29] that's bizarre.. i don't touch them! :) [20:11:45] not client-side - lemme clear my varnish cache to be sure [20:12:17] so yeah, not caching [20:12:20] but https://gerrit.wikimedia.org/r/#/c/58824/3 makes no changes to the pageimage itself! :D [20:12:25] still seeing the problem [20:12:30] could be related to other css around it [20:12:39] awjr: let me check we are talking about the same thing [20:12:42] k [20:12:58] in master the watch list thumbs ARE different from nearby thumbs. Patchset https://gerrit.wikimedia.org/r/#/c/58824/3 doesn't introduce it. [20:15:22] yes [20:15:29] cool [20:15:35] i''m fixing up the page images now [20:15:42] groovy [20:15:45] jdlrobson: i sent ya some screenshots [20:15:56] thanks awjr [20:16:03] np jdlrobson [20:16:15] are you making changes in https://gerrit.wikimedia.org/r/#/c/58824/3 or are you doing a separate patch? [20:16:45] jdlrobson: ^? if the latter, im ok to merge https://gerrit.wikimedia.org/r/#/c/58824/3 [20:18:25] New patchset: awjrichards; "Story 430: Promote nearby to beta" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58842 [20:18:54] New patchset: Jdlrobson; "Unsquish pageimages on nearby" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59002 [20:19:24] New patchset: Jdlrobson; "fix jshint error" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59003 [20:19:37] awjr: separate patch as it is unrelated ^ [20:20:05] Change merged: Jdlrobson; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59003 [20:20:50] Change merged: jenkins-bot; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58824 [20:21:17] Change merged: jenkins-bot; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58842 [20:23:16] so MaxSem since jcmish is out… do you want some help wrangling for some testing on testwiki? [20:24:07] would be great [20:24:44] MaxSem: do you have an idea of the specific devices you want to test against, or do you need some help pulling that together too? [20:24:52] Change merged: jenkins-bot; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59002 [20:25:12] everything but latest android/iphone:) [20:25:30] heh ok [20:25:50] jdlrobson: do you have some time today/monday to help test devices against the wgMFVaryResources change that's up on testwiki? [20:26:07] this is to make sure that not varying page html on device will still work for the different device types [20:26:20] awjr: sure although my array of devices is not as extensive as you think - the ones we need to worry about are the ones we don't own [20:26:39] we need some basic testing before unleashing volunteers on it [20:26:58] jdlrobson, MaxSem if you guys can compile a list of the devices we don't have to test against but should test against and mail it to mobile-tech, hopefully jcmish can help with that once she's back on monday [20:27:16] awjr: that list would be ridiculously long! :) [20:27:29] awjr: this is why i think crowd sourcing is the best approach [20:27:38] * YuviPanda plugs mobile-l, then slinks away [20:27:39] i can do basic testing but it's useless compared to doing that [20:27:47] jdlrobson: well, we have a finite list of X-Device types; is it not enough to test against one device from each type? [20:28:02] jdlrobson: for sure; but like max said, quick sanity test before we push it live for crowd-testing [20:28:19] (we're going to make it live on mediawiki.org only to start) [20:28:27] ok, iOS commons is submitted for review again [20:28:32] and available on test flight for betaers [20:28:51] and this time i gave them credentials and uploaded a couple stub photos into their account :) [20:29:05] brion: there's someone on commons-l who had problems with testflight [20:29:09] brion: are you on commons-l? [20:29:11] brion: also, YAY! :) [20:29:16] brion: now if those files get deleted... [20:29:17] i'm not, i should join [20:29:28] YuviPanda: was it florence? i sent her a message with some directions [20:29:28] brion: do :) [20:29:53] brion: y'know, you could've sent that 1.0 email to mobile-l instead of mobile-tech :) [20:29:57] jdlrobson: are there specific devices you're concerned about or is just covering the device types from our DeviceDetection sufficient? [20:29:59] brion: yeah, florence [20:30:13] YuviPanda: i will too, thx for the reminder :D [20:30:43] brion: :) [20:30:54] brion: I forwarded a message from past-you to mobile-tech [20:30:56] awjr: i'm not too concerned about anything really. I'm very optimistic - I'm just worried about very obscure phones i've never heard of :) [20:31:10] oh, well we can unearth that in crowd-testing [20:31:22] i dont think we need to worry about edge cases for rolling this out to mw.o [20:32:47] MaxSem: can you mail out to mobile-tech about our rollout plan? and jdlrobson can you reply with the devices we can test testwiki against for sanity before rolling this out next tues? i can talk to michelle about coordinating broader testing once it's out on mw.o [20:34:23] awjr: why not to mobile-l in the spirit of YuviPanda :) [20:34:30] even better jdlrobson [20:34:45] whee [20:34:46] I think I already explained my plan: in-house testing -> enable on mw.o -> volunteer testing --> everywhere (or how Mark will decide) [20:35:31] MaxSem: yeah but nobody other than us are aware of it :) [20:35:53] and the directors since i mentioned it in the deployment meeting [20:36:19] the rest of the team should be aware of it - plus, if it didn't happen on the mailing list, it didn't happen at all. [20:36:23] YuviPanda: http://lists.wikimedia.org/pipermail/mobile-l/2013-April/005939.html [20:36:33] wheee [20:37:14] New patchset: Jdlrobson; "Bug 44959: Allow overlay heading title to span 2 lines" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58725 [20:38:54] jdlrobson: im getting a js error with https://gerrit.wikimedia.org/r/#/c/58847/2 - Uncaught TypeError: Cannot read property 'pages' of undefined [20:39:23] errgg did you set your nearby url to wikipedia? [20:39:37] brion: i'm rewriting [[Commons:Mobile app]], filling it with screenshots [20:39:42] awjr: try rebasing [20:39:46] brion: i'm adding a Features section with subsections for Android / iOS [20:39:53] brion: can you fill the iOS sections ometime? [20:39:56] hehehe i really read that first as 'try freebasing' [20:40:03] it's probably using the old nearby url variable [20:40:31] awjr: thanks for taking the time to code review today i really appreciate it :) [20:40:47] np [20:41:10] oh yeah, rebase is all wonky :-/ [20:41:13] New patchset: awjrichards; "Story 419: Nearby distance enhancements" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58847 [20:41:54] good call jdlrobson, looks good now [20:42:52] Change merged: jenkins-bot; [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/58847 [20:43:30] brion: also did you hear of the amazing plot to fix the brokenness of UploadCampaigns? [20:43:51] \o/ [20:47:46] New patchset: Jdlrobson; "Story 426: Use file name for images with descriptions with templates" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/56613 [20:47:50] New patchset: Jdlrobson; "Story 426: Use file name for images with descriptions with templates" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/56613 [21:06:50] alright im off to bring my pup to the vet - hopefully back in an hour or so [21:27:30] New patchset: Jdlrobson; "Bug 46906: Load mobile options at top of page" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59012 [21:27:30] New patchset: Jdlrobson; "On settings page allow touchstart to move checkbox" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59013 [21:52:37] New patchset: Jdlrobson; "Link to php guidelines in README" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/59015 [22:08:20] ei, dan & kul are alive! [22:08:49] they just made it to the hotel, exhausted, but apparently bearing good news :) [22:10:28] \o/ [22:14:47] Maryana: is there anyway we can get vibha on irc more? [22:14:58] highly doubtful [22:15:06] she's on gchat most of the time [22:15:09] :( [22:15:12] * jdlrobson hates gchat [22:15:22] can you ping her and see if i can bug her quickly? [22:15:24] heh. well, i'm sure vibha hates irc [22:15:30] isn't irc kind of corporate standard? [22:15:37] "corporate" [22:15:38] hehe [22:15:42] THE MAN forces us to use irc [22:15:50] lol [22:15:57] it's great.. if you're a developer [22:15:57] which one?:) [22:16:18] * MaxSem sends Maryana a bunch of /commands [22:16:30] IRC is a power tool [22:16:37] yeahhhhh. that is not so intuitive for people who don't write code [22:16:44] or were born after 1972 [22:16:51] ;) [22:16:54] Maryana: if she hates the usual tools, get her to try LimeChat [22:16:58] * YuviPanda hates GChat too [22:17:15] YuviPanda, just like everyone else [22:17:20] gchat keeps turning on on my phone [22:17:27] occasionally i get a ping and i'm all "what? that's on?" [22:17:45] brion: at least if they had implemented it properly, it's not draining your battery :) [22:17:53] ehh [22:18:11] oh man did you see the new permissions on the android Facebook app? [22:18:25] it seems like they stuck Facebook home as an optional feature of Facebook app or something [22:18:31] so it demands all these weird perms [22:19:57] brion: yeah, I commented [22:19:59] :) [22:20:11] brion: it'll next want your root password ;) [22:20:15] i love complaining about Facebook on facebook [22:20:20] brion: i mean, really? permission to overwite on top of other apps? [22:20:27] must be for the 'chat heads' thing [22:20:41] for the paranoid, that'll be 'hey let us replace your password looking textbox with our own'... [22:20:50] ya :( [22:21:27] same reason firefox and chrome give you DIRE WARNINGS when a site enters fullscreen mode [22:21:34] but they only do it when it actually tries [22:21:50] not just the first time you go there, "hey someday we might need THIS so pre-approve us forever please" [22:23:32] brion: yeah! [22:24:03] brion: I've merged like, 9 patches and CR'd about 7 in the last two days, all completely unrelated to mobile [22:24:05] this feels good :) [22:24:19] :D [22:24:29] i'll have to put in some more review time myself, i've been lazy lately [22:24:54] brion: :) yes [22:30:45] AwkwardInterview++ [22:45:18] jdlrobson: the story # referenced in this patchset is incorrect: https://gerrit.wikimedia.org/r/#/c/56613/5 and im having a hard time finding the actual story card - do you know what it is by chance? [22:45:27] * jdlrobson looks [22:45:58] New patchset: Jdlrobson; "Story 436: Use file name for images with descriptions with templates" [mediawiki/extensions/MobileFrontend] (master) - https://gerrit.wikimedia.org/r/56613 [22:45:59] fixed :) [22:46:02] 10 out [22:46:04] ahha thanks [22:46:16] it's an old patch set from juliusz repurposed as an upcoming story in the next iteration ;-) [22:46:22] yeah :) [22:51:02] AwkwardInterview: brion I just filled out http://commons.wikimedia.org/wiki/Commons:Mobile_app [22:51:07] and am going to put a word on the VP [22:51:31] feel free to edit if I had missed something there [22:51:51] \o/ [22:52:01] ooh i'll make new videos monday [22:52:07] brion: also Android crossed 1k uploads ;) [22:52:12] awesome [22:52:27] brion: and still at 13% deletion rate :) [22:52:28] so yay [22:52:34] though that is a bit too high, IMO [22:52:34] hehe [22:52:36] because of our test uploads [22:52:37] that's not bad though [22:52:45] better than 80% [22:55:08] brion: heh, yeah [22:55:15] brion: the web category now has ~500 uploads [22:55:49] i've got more people signing up for the iOS beta team but they're not connecting their devices [22:56:24] wonder if test flight could make that more ….. easier to do [22:56:31] yeah [22:56:38] i need to check testflight's android thing [22:57:21] honestly the store works great for betas on android [22:59:31] brion: true but the problem is that i need to keep it as a different package [22:59:47] brion: so that means i need to have another branch with a massive file move /rename thing [22:59:47] mm yes [22:59:48] :( [22:59:49] like, the folder structure == package name thing fucks shit up [22:59:57] brion: and worst part? I'll need to change the *imports* as well! [23:00:03] could you make a script that just searches and replaces? :D [23:00:07] so everytime I touch the imports, it'l cause a merge conflict [23:00:12] could... [23:00:13] :) [23:02:15] brion: did you catch vibha about the design for the tutorial stuff? [23:02:25] nope [23:07:51] brion: ok [23:09:08] jdlrobson: have you had a chance to do some sanity checking of various devices on test.m.wikipedia.org ? [23:09:23] i've started awjr slowly [23:09:29] cool jdlrobson, thanks [23:16:12] Maryana: got a moment? [23:22:12] Maryana: http://dpaste.de/57TUr/ thoughts? I'm posting that to the VP [23:23:12] "ago(" [23:24:57] fixed [23:25:10] anything else? [23:29:13] you randomly capitalized "Test" [23:29:40] "Looking forward to help increase the number of undeleted mobile uploads to commons" sbm [23:31:53] brion: do you have a link i can use for test flight beta entry on the about page? been poking around and not sure which one to use [23:31:57] legoktm: sbm? [23:32:12] http://www.urbandictionary.com/define.php?term=sbm [23:32:14] legoktm: the capitalization i find is 'Test images' [23:32:16] and being nouny... [23:32:17] the first one. [23:32:31] haha [23:32:44] mhurd: try the one in my last email to commons-l, lemme bring it up [23:32:51] oh man why did i scroll down [23:32:57] legoktm: imma post. [23:33:01] http://lists.wikimedia.org/pipermail/commons-l/2013-April/006791.html [23:33:05] lgtm +1 [23:33:07] http://tflig.ht/Zl9Ef7 [23:33:20] posting on VP then going to sleep is the new deploying on a friday night and heading out to get sloshed :P [23:33:22] beware that's a lowercase L not an uppercase I [23:33:22] awesome! [23:33:35] YuviPanda: hehehe [23:33:44] pitchforks, ho! [23:33:54] indeedy! [23:34:50] well that's done [23:34:52] gnite everyone [23:35:19] nini [23:35:25] gnite [23:35:26] see ya! [23:35:28] brion: can you start a small thread with luis about going Apache / non-GPL for the iOS app? [23:35:38] should be a quick okay, then we can move it :) [23:35:52] * brion adds it to the list [23:36:16] it's on the board ;) [23:36:19] okay ow to really sleep [23:36:47] :D [23:39:11] YuviPanda: you know this skin i've been designing? [23:39:42] it seems to give desktop a beta/stable mode as a side effect o_O [23:39:49] alpha/beta/stable