[00:06:31] (03PS1) 10Legoktm: Add "composer test" command to run phpcs, and fix warnings [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196972 [00:07:22] (03CR) 10Legoktm: "check experimental" [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196972 (owner: 10Legoktm) [00:09:35] (03CR) 10Legoktm: "Once this is merged I can make phpcs voting (through the php-composer-test job) if the maintainers want, otherwise I'll disable the deprec" [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196972 (owner: 10Legoktm) [00:23:25] (03PS2) 10Florianschmidtwelzow: Hygiene: Use ResourceLoader::isModuleRegistered() [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/196968 (owner: 10Legoktm) [00:23:30] (03CR) 10Florianschmidtwelzow: [C: 032] Hygiene: Use ResourceLoader::isModuleRegistered() [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/196968 (owner: 10Legoktm) [00:24:16] (03CR) 10Legoktm: "This isn't "Hygiene"...it's performance." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/196968 (owner: 10Legoktm) [00:24:41] (03Merged) 10jenkins-bot: Hygiene: Use ResourceLoader::isModuleRegistered() [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/196968 (owner: 10Legoktm) [00:24:57] (03CR) 10Florianschmidtwelzow: "But we have no Performance tag and it doesn't change anything visible to the user :/" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/196968 (owner: 10Legoktm) [00:26:02] legoktm: ^ it looks like a Hygiene for me :/ (see, e.g., other changes tagged as hygiene in MobileFrontend) [00:27:06] https://en.wiktionary.org/wiki/hygiene "Cleanliness" [00:27:12] this is much more than that :P [00:27:16] anyways, thanks for the +2 :) [00:27:51] hmpfh [00:27:58] sorry :) [00:28:16] and np, i should wait next time when i change the commit message :) [00:31:23] nah, I don't care that much :P [00:35:15] (03CR) 10Dbrant: [C: 032] Fix issue where the bottom portion of About screen was white in dark mode [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/196586 (owner: 10BearND) [00:36:31] (03CR) 10Dbrant: [C: 032] Light theme chooser button was not labelled on 2.3 [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/196705 (owner: 10BearND) [00:39:34] (03CR) 10Dbrant: [C: 032] Update store description string. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/196728 (owner: 10Deskana) [00:40:47] FlorianSW, why do you tag commits like that? :/ [00:40:57] (03PS1) 10Dbrant: Cache pages on disk instead of ram. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/196974 [00:41:19] (03CR) 10jenkins-bot: [V: 04-1] Cache pages on disk instead of ram. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/196974 (owner: 10Dbrant) [00:44:55] (03PS2) 10Dbrant: [WIP] Cache pages on disk instead of ram. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/196974 [01:08:17] (03PS1) 10Niedzielski: Remove executable bit on image file. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/196979 [01:09:03] (03CR) 10Niedzielski: "Smallest possible change I could make to get my feet wet." [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/196979 (owner: 10Niedzielski) [01:33:41] (03PS2) 10Jforrester: Add "composer test" command to run phpcs, and fix warnings [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196972 (https://phabricator.wikimedia.org/T90943) (owner: 10Legoktm) [01:33:49] (03CR) 10Jforrester: [C: 031] Add "composer test" command to run phpcs, and fix warnings [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196972 (https://phabricator.wikimedia.org/T90943) (owner: 10Legoktm) [02:06:58] (03PS3) 10Dbrant: [WIP] Cache pages on disk instead of ram. [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/196974 [08:31:21] (03PS1) 10Mhurd: Some small cleanups. [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/197005 [08:37:52] (03PS1) 10Mhurd: Fix for image saving bug. [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/197006 [09:17:36] morning [09:30:23] good morning [09:45:50] (03CR) 10Phuedx: [C: 032] Fix styling regression in new editor tutorial [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/196324 (https://phabricator.wikimedia.org/T91141) (owner: 10Bmansurov) [09:45:59] hey joakino [09:46:37] hi phuedx [09:47:06] (03Merged) 10jenkins-bot: Fix styling regression in new editor tutorial [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/196324 (https://phabricator.wikimedia.org/T91141) (owner: 10Bmansurov) [09:47:39] 6Mobile-Web, 7Design: Styling regression in new editor tutorial - https://phabricator.wikimedia.org/T91141#1120872 (10phuedx) [09:47:47] 6Mobile-Web, 7Design: Styling regression in new editor tutorial - https://phabricator.wikimedia.org/T91141#1120874 (10phuedx) 5Open>3Resolved a:3phuedx [10:00:00] how was yer weekend joakino ? [10:00:40] phuedx: pretty lazy, I finished rogue legacy and watched kingsman [10:00:44] what about yours phuedx [10:00:50] diy [10:01:03] lots of diy [10:01:19] and coffee shop research [10:01:47] so not so lazy [10:03:52] oh and generally being bloody cold [10:03:59] finally getting the new boiler installed today [10:04:39] but no coding [10:04:47] very little technical reading [10:05:04] just hanging out with my family, building kitchen cabinets and trying to find shelving to fit a hallway cupboard [10:05:07] was nice [10:09:12] logs! [10:14:00] joakino: is kingsman any good? [10:14:31] phuedx: a real family man :D [10:14:52] phuedx: i found it funny and entertaining [10:14:59] good comedy [10:15:17] isn't it supposed to be an action thing? [10:15:54] yep but it's comedy too [10:16:04] according to imdb Action, Adventure, Comedy [10:20:14] joakino: i'm also going to buy an exercise bike [10:20:30] (because i don't have enough time to go to the gym) [10:23:11] phuedx: cool! [10:23:14] how much are those? [10:23:39] for 5-8 per week usage: 150-200 new, 100 used [10:24:06] an hour a day is achievable –– because i wake up at 6 to feed harry, then he goes to sleep for an hour [10:24:26] i reckon i could get an hour in (when i'm capable of cycling for an hour…) [10:29:07] phuedx: that's a good price [10:29:22] joakino: i think so [10:29:32] i usually do 40 minutes sessions (30 minutes intensive, 10 cool down) [10:29:50] i get bored if i have to stay more time [10:30:03] inorite [10:30:10] podcasts/tech talks a-plenty [10:31:07] still 40 minutes a day is great [10:31:14] i used to cycle everywhere [10:31:18] now i don't [10:31:19] :( [10:32:51] 6Mobile-Web, 5Patch-For-Review: MobileFrontend and $wgUseFileCache - https://phabricator.wikimedia.org/T78021#1120983 (10phuedx) [179666](https://gerrit.wikimedia.org/r/179666) was cherry-picked to REL1_24 by @Florian in [194527](https://gerrit.wikimedia.org/r/194527). [10:33:18] 6Mobile-Web: MobileFrontend and $wgUseFileCache - https://phabricator.wikimedia.org/T78021#1120985 (10phuedx) [10:33:27] 6Mobile-Web: MobileFrontend and $wgUseFileCache - https://phabricator.wikimedia.org/T78021#1120986 (10phuedx) 5Open>3Resolved [11:39:58] (03PS1) 10Jhernandez: Default to 50 items when seeing collection [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197021 (https://phabricator.wikimedia.org/T90903) [11:57:30] joakino, hi [11:58:59] hi yurik [11:59:13] joakino, i think paging is really needed pretty soon ) [11:59:25] yep, will tackle next sprint :D [11:59:36] joakino, did you see my thoughts about the model? [12:01:21] yurik: yes, i think we need to separate the style issues (which are personal preference) and the dumb stuff we are doing by querying that much, i think they are orthogonal, I'm trying to optimize some newFromApi methods to be smarter [12:02:05] joakino, the problem with optimizing model (newFromApi) is that you are basically replicating all the API features in a separate function [12:02:17] our newFromApi methods are doing stupid shit, and the onMakeGlobalVariables method is doing a looooooooooooot of dumb queries, but that's fixable without going full imperative C :p [12:02:39] yurik: i'm not, i'm querying the api better, because it was querying it wrong [12:03:19] yurik: I'm with Collection::newFromApi which for some reason was requesting the whole lists of the user, when it can call the api with just lstids [12:03:29] api filters, no need to do it on php [12:04:02] later i'll go to the onMakeGlobalVariables and make a method that will query the api correctly and as little as possible to get everything needed [12:04:23] joakino, onMakeGlobalVariables should not get everything [12:04:44] it only needs one thing - does the current page belong to any lists (true/false) [12:04:46] yurik: not everything, "everything needed" [12:04:59] yurik: that and the list of collections of the logged in user [12:05:06] why list? [12:05:39] i think to avoid ajax calls when watching it [12:06:05] joakino, i don't think its a good strategy here - a very quick AJAX call on user click is ok IMO [12:06:21] what you are doing otherwise is a massive extra SQL work for EVERY page load [12:06:35] yurik: if that's too intensive let's open a bug [12:06:37] whereas the user would click the star once in a blue moon [12:06:44] its open [12:08:05] hmm.. at least i thought it was open... loking [12:08:24] yurik: i've created this https://phabricator.wikimedia.org/T92821 [12:08:29] not sure if there's any other issue [12:08:53] but yep, seems like a sensible idea worth doing [12:09:01] ok, will dup if i find the old one [12:09:21] basically its a difference of doing a table join vs doing an "exists" lookup [12:09:36] yurik: i'm fine with that :) [12:09:50] yurik: with what you're saying [12:10:08] joakino, ok, in that case, my present to you :) https://gerrit.wikimedia.org/r/#/c/196739/ [12:13:42] yurik: cool! will have a look later [12:13:44] i'm about to leave [12:16:03] yurik: is it possible that querying action=query&list=lists with a lstids that doesn't exist gives me always the watchlist? [12:16:29] joakino, it shouldn't i think [12:16:49] hmm [12:16:58] joakino, send me the api req [12:17:05] and the state of the DB [12:17:37] yurik: so if i do this query http://localhost:8080/w/api.php?action=query&list=lists&format=jsonfm&lstprop=label%7Cdescription%7Cpublic%7Cimage%7Ccount&lstids=8 [12:17:52] yurik: when in an incognito window returns the proper list (which is public) [12:18:04] yurik: but when logged in with the owner it's giving me the watchlist [12:18:22] yurik: should I add the lstowner parameter maybe? [12:18:27] investigating [12:18:49] joakino, so your goal is to get info about the specific list, right? [12:18:53] if I add lstowner when logged in, it gives me both 8 and the watchlist [12:18:56] yurik: yep! [12:19:24] joakino, could you pull my patch and retry it? i might have fixed that issue as part of that patch [12:19:39] i will recheck. [12:19:47] mergeing of watchlist and other lists is tricky [12:19:56] might have bugs lurking [12:20:34] yurik: k [12:21:58] yurik: same behavior [12:22:00] :( [12:22:15] gotta go, i'll file a more specific bug later [12:22:21] yurik: i'll check that patch later [12:22:34] ok [13:23:15] joakino, found a bug, need to think how to solve it better. Should be done in a bit [13:23:25] joakino, pls file [14:23:40] (03CR) 10Yurik: [C: 032] Default to 50 items when seeing collection [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197021 (https://phabricator.wikimedia.org/T90903) (owner: 10Jhernandez) [14:24:47] (03Merged) 10jenkins-bot: Default to 50 items when seeing collection [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197021 (https://phabricator.wikimedia.org/T90903) (owner: 10Jhernandez) [15:14:33] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Setup iOS build server and distribute alpha builds internally - https://phabricator.wikimedia.org/T91786#1121514 (10Fjalapeno) [15:16:52] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Setup iOS build server and distribute alpha builds internally - https://phabricator.wikimedia.org/T91786#1121527 (10Fjalapeno) [15:26:36] 6Mobile-Web: The mobile Web wikicode editor loses the user-entered text - https://phabricator.wikimedia.org/T88949#1121563 (10Nnemo) I am not sure about what exactly some people want as a new task. It seems to me that this new task would just describe one technical solution to the functional problem described he... [15:27:48] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Set up XCode dashboard to run unit tests on master branch - https://phabricator.wikimedia.org/T91788#1121567 (10Fjalapeno) Note: We are no longer using Xcode bots, so this task is slightly different than before. [15:32:20] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Set up XCode dashboard to run unit tests on master branch - https://phabricator.wikimedia.org/T91788#1121580 (10Fjalapeno) [15:33:24] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Run unit tests before builds and save build reports for viewing - https://phabricator.wikimedia.org/T91788#1121585 (10Fjalapeno) [15:33:43] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Run unit tests before builds and save build reports for viewing - https://phabricator.wikimedia.org/T91788#1096355 (10Fjalapeno) [15:48:05] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: [Blocked] Build and distribute beta build to external testers weekly - https://phabricator.wikimedia.org/T91791#1121649 (10Fjalapeno) [15:56:42] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Build and test all Gerrit patch sets - https://phabricator.wikimedia.org/T91790#1121677 (10Fjalapeno) [15:56:43] (03PS2) 10Phuedx: [WIP] Footer changes [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/195552 [16:06:05] 6Mobile-Web: The mobile Web wikicode editor loses the user-entered text - https://phabricator.wikimedia.org/T88949#1121713 (10TheDJ) ehm, why did MF not reuse http://git.wikimedia.org/blob/mediawiki%2Fcore.git/093bb2e23e7a984eee9c32be79058c6e6974a740/resources%2Fsrc%2Fmediawiki%2Fmediawiki.confirmCloseWindow.js ? [16:12:07] 10Wikipedia-App-iOS-App: [4.0.7.5] Lead Image & Image Gallery features broken for migrated articles - https://phabricator.wikimedia.org/T92697#1121730 (10Etonkovidova) It seems that opening saved/recent pages the second etc time displays the leading image (iPad mini 8.2 with Beta 4.0.7.5 ). [16:22:33] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App: Portal template displays an additional element instead of being rendered as just links - https://phabricator.wikimedia.org/T92659#1121755 (10bearND) [16:27:58] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Setup iOS build server and distribute alpha builds internally - https://phabricator.wikimedia.org/T91786#1121774 (10BGerstle-WMF) @Fjalapeno this is a great breakdown. I need to write some code now, but I'd like to come back and separate these into subtasks w... [16:30:02] (03PS1) 10BearND: Set text color in error popups for 2.3 [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/197077 [16:30:53] 10Wikipedia-App-iOS-App: [4.0.7.5] Lead Image & Image Gallery features broken for migrated articles - https://phabricator.wikimedia.org/T92697#1121780 (10BGerstle-WMF) @Etonkovidova that sounds inconsistent with what I noticed, which is that no matter how many times I opened the article from saved/recent that th... [16:31:54] 10Wikipedia-App-iOS-App, 3Mobile App Sprint 52 - iOS: 4.0.6.3 Image Gallery - some images displayed in low resolution - https://phabricator.wikimedia.org/T91124#1121784 (10BGerstle-WMF) I had started working on a fix which involved paginating the "imageinfo" requests (which return metadata and a high-res URL).... [16:35:49] HOT WATER!!!!!?!?!?!?@!4098721098 [16:47:08] (03PS1) 10Jhernandez: WIP: A more optimal Collection::newFromApi [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197085 [16:47:10] (03CR) 10jenkins-bot: [V: 04-1] WIP: A more optimal Collection::newFromApi [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197085 (owner: 10Jhernandez) [16:49:20] (03PS3) 10Phuedx: Footer changes [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/195552 [17:01:30] yurik: I was backing your proposal of onArticleDisplay (instead of onMakeGlobalVariables) just querying the api for if the article is contained in lists, and then query the api in the ui via ajax for the collections list (instead of on the server) [17:02:16] joakino: thanks! my connection is crappy, could hear every other word. Even here I'm through a web IRC instead of via my app (( [17:02:48] yurik: no prob, that's bad :( [17:02:53] yurik: where are you exactly? [17:02:55] heh [17:02:59] st. petersburg [17:03:15] cool [17:03:20] when i work from home, its perfect, but when i'm in a coffee shop, its a gamble [17:03:34] i don't have notifications here it seems [17:03:52] so best way to ping me is via google messenger [17:04:04] but i will try to peek in here [17:08:20] 10Wikipedia-App-Android-App, 3Mobile-App-Sprint-53-Android: Login/create account/captcha buttons not visible in 2.3 - https://phabricator.wikimedia.org/T92850#1121932 (10bearND) 3NEW [17:10:55] 6Mobile-Web: No entry point for commons category - https://phabricator.wikimedia.org/T92687#1121945 (10KLans_WMF) p:5Triage>3Normal [17:11:57] 6Mobile-Web: Not obvious when in alpha/beta - https://phabricator.wikimedia.org/T92679#1121950 (10KLans_WMF) a:3Maryana [17:12:16] 10Wikipedia-App-Android-App, 3Mobile-App-Sprint-53-Android: Login/create account/captcha buttons not visible in 2.3 - https://phabricator.wikimedia.org/T92850#1121954 (10bearND) [17:12:28] 6Mobile-Web: Not obvious when in alpha/beta - https://phabricator.wikimedia.org/T92679#1117970 (10KLans_WMF) Maryana is going to storify this [17:12:57] 6Mobile-Web: Search workflow in alpha is terrible for non-JavaScript users - https://phabricator.wikimedia.org/T92675#1121958 (10KLans_WMF) a:3Maryana [17:14:12] yurik: hangout to the wikimedia address? [17:14:31] 10Wikipedia-App-Android-App, 3Mobile-App-Sprint-53-Android: Login/create account/captcha buttons not visible in 2.3 - https://phabricator.wikimedia.org/T92850#1121967 (10bearND) a:3bearND [17:16:50] 10Gather, 3Gather Sprint D_: Article reader sees message/guidance about creating a collection so that they know the feature exists and the entry point - https://phabricator.wikimedia.org/T91015#1121972 (10JKatzWMF) [17:17:00] 10Gather, 3Gather Sprint D_: Loading lists of user in every page load is too intensive - https://phabricator.wikimedia.org/T92821#1121973 (10Jhernandez) [17:17:01] 3Gather Sprint C_, 5Patch-For-Review: Performance: onMakeGlobalVariablesScript causes too many internal hits to API - https://phabricator.wikimedia.org/T92580#1121975 (10Jhernandez) [17:17:03] dr0ptp4kt: you standing? [17:17:29] 10Gather: Loading lists of user in every page load is too intensive - https://phabricator.wikimedia.org/T92821#1121184 (10Jhernandez) [17:17:44] 10Wikipedia-App-Android-App, 3Mobile-App-Sprint-53-Android: Cache visited pages in filesystem instead of ram. - https://phabricator.wikimedia.org/T89826#1121978 (10Dbrant) [17:17:48] 10Wikipedia-App-Android-App, 3Mobile-App-Sprint-53-Android: Text in error popups not visible in 2.3 - https://phabricator.wikimedia.org/T92851#1121979 (10bearND) 3NEW a:3bearND [17:17:53] 10Gather: Loading lists of user in every page load is too intensive - https://phabricator.wikimedia.org/T92821#1121184 (10Jhernandez) Let's continue talking in the older task T92580 [17:18:18] (03PS2) 10Jhernandez: WIP: A more optimal Collection::newFromApi [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197085 (https://phabricator.wikimedia.org/T92580) [17:18:20] (03CR) 10jenkins-bot: [V: 04-1] WIP: A more optimal Collection::newFromApi [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197085 (https://phabricator.wikimedia.org/T92580) (owner: 10Jhernandez) [17:19:01] (03PS2) 10BearND: Set text color in error popups for 2.3 [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/197077 (https://phabricator.wikimedia.org/T92851) [17:19:38] 10Gather, 3Gather Sprint D_: Article Deletion handling - https://phabricator.wikimedia.org/T90992#1122009 (10JKatzWMF) [17:20:28] 10Gather, 3Gather Sprint C_: Watchstar behaviour - https://phabricator.wikimedia.org/T91674#1093001 (10JKatzWMF) [17:25:19] 3Gather Sprint D_: Event logging for Gather onboarding - https://phabricator.wikimedia.org/T92742#1122021 (10JKatzWMF) [17:25:35] 10Gather: Creating EL Schema - https://phabricator.wikimedia.org/T90987#1122032 (10JKatzWMF) [17:26:03] 3Gather Sprint C_, 5Patch-For-Review: Performance: onMakeGlobalVariablesScript causes too many internal hits to API - https://phabricator.wikimedia.org/T92580#1122042 (10Jhernandez) I have a WIP patch ^^^ that improves Collection::newFromApi which should improve loading Special:Gather/User/XXX but it doesn't w... [17:26:28] joakino: Do you think this bug could have anything to do with Gather: https://phabricator.wikimedia.org/T92657 ? It’s difficult to debug since it doesn’t occur with debug=true and so far no one can reproduce it locally (although it happens on beta labs, test.wiki, and en.wiki). [17:26:32] 3Gather Sprint D_: Event logging for Gather onboarding - https://phabricator.wikimedia.org/T92742#1119233 (10JKatzWMF) [17:27:21] kaldari: we are not deployed yet on en.wiki so that's impossible [17:27:50] 10Gather, 3Gather Sprint D_: Event logging for hamburger "collections" link - https://phabricator.wikimedia.org/T92855#1122056 (10JKatzWMF) 3NEW [17:30:45] wow kaldari only on debug=false [17:30:46] crap [17:34:45] 10Gather: Loading lists of user in every page load is too intensive - https://phabricator.wikimedia.org/T92821#1122081 (10Yurik) @Jhernandez, this is really not a dup but should be a performance tracking bug to do overall perf optimization. I will reopen a general tracker. [17:35:28] phuedx: is the hangout working? [17:35:42] joakino: yeah pour moi [17:35:46] ok now phuedx thx [17:36:12] (03CR) 10Bmansurov: [C: 04-1] "Also wikigrok C doesn't load the next page automatically." (032 comments) [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/195552 (owner: 10Phuedx) [17:37:07] 10Gather: Tracking: Performance - https://phabricator.wikimedia.org/T92857#1122082 (10Yurik) 3NEW [17:38:13] 10Gather: Tracking: Performance - https://phabricator.wikimedia.org/T92857#1122093 (10Yurik) [17:38:40] 10Gather: Only call API dynamically except for special:gather - https://phabricator.wikimedia.org/T92711#1122098 (10Yurik) [17:38:41] 10Gather: Tracking: Performance - https://phabricator.wikimedia.org/T92857#1122082 (10Yurik) [17:40:39] 6Mobile-Web: Special:MobileOptions throwing JS error from mobile.watchstar in stable - https://phabricator.wikimedia.org/T92657#1122104 (10Jhernandez) @kaldari I think that watchstar init.js was split recently, in there it is the ` TypeError: M.getCurrentPage is not a function` that is being called, line 23. It... [17:41:10] coreyfloyd: bgerstle able to get on a video call? [17:41:23] sure [17:41:31] (03CR) 10Phuedx: Footer changes (032 comments) [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/195552 (owner: 10Phuedx) [17:41:33] dr0ptp4kt: in the same room [17:41:37] dr0ptp4kt: as standup [17:41:43] (03PS4) 10Phuedx: Footer changes [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/195552 [17:45:44] 3Gather Sprint D_: Optimize Special:Gather/User/XXX loading - https://phabricator.wikimedia.org/T92862#1122137 (10Jhernandez) 3NEW a:3Jhernandez [17:46:01] (03PS3) 10Jhernandez: WIP: A more optimal Collection::newFromApi [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197085 (https://phabricator.wikimedia.org/T92862) [17:46:03] (03CR) 10jenkins-bot: [V: 04-1] WIP: A more optimal Collection::newFromApi [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197085 (https://phabricator.wikimedia.org/T92862) (owner: 10Jhernandez) [17:48:13] (03PS1) 10Bmansurov: Add dev-scripts for easy development [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/197092 [17:52:56] 6Mobile-Web: Search workflow in alpha needs non-JavaScript fallback - https://phabricator.wikimedia.org/T92675#1122168 (10Maryana) [17:54:07] 3Gather Sprint D_: Api bug when loading action=query&list=lists&lstids=X and logged in - https://phabricator.wikimedia.org/T92865#1122170 (10Jhernandez) 3NEW [17:54:22] 3Gather Sprint D_: Api bug when loading action=query&list=lists&lstids=X and logged in - https://phabricator.wikimedia.org/T92865#1122177 (10Jhernandez) It would be great to hear if anybody else can reproduce [17:54:44] jdlrobson: hi :) Have you a minute to talk? [17:54:47] damn [17:55:06] 3Gather Sprint D_, 5Patch-For-Review: Optimize Special:Gather/User/XXX loading - https://phabricator.wikimedia.org/T92862#1122178 (10Jhernandez) [17:55:06] 3Gather Sprint D_: Api bug when loading action=query&list=lists&lstids=X and logged in - https://phabricator.wikimedia.org/T92865#1122179 (10Jhernandez) [17:55:23] 3Gather Sprint D_, 5Patch-For-Review: Optimize Special:Gather/User/XXX loading - https://phabricator.wikimedia.org/T92862#1122137 (10Jhernandez) Patch in WIP until the blocker is solved and I can actually test this. [17:55:38] jdlrobson: did you receive my message? (hi :) Have you a minute to talk?) [18:00:08] 6Mobile-Web: Search workflow in alpha needs non-JavaScript fallback - https://phabricator.wikimedia.org/T92675#1122184 (10Maryana) @jdlrobson Let's uphold our end of the bargain on good communication on Phab & please not title bugs with "X is terrible". {F95512} ^ I think this is still what we want for the non... [18:00:15] (03PS1) 10BearND: Remove (R) from wordmark in ShareAFact image [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/197098 [18:02:08] Deskana|Away: ^ [18:20:20] (03PS2) 10Krinkle: Register hogan.js via module name 'mediawiki.template.hogan' [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/195803 [18:20:39] (03CR) 10Krinkle: "Just an example commit to show the idea. Feel free to fork/amend." (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/195803 (owner: 10Krinkle) [18:22:36] 10Gather, 3Gather Sprint D_: Tracking: Performance - https://phabricator.wikimedia.org/T92857#1122322 (10Jdlrobson) [18:23:09] 3Gather Sprint D_, 5Patch-For-Review: Performance: onMakeGlobalVariablesScript causes too many internal hits to API - https://phabricator.wikimedia.org/T92580#1122323 (10Jdlrobson) [18:23:18] 10Gather, 3Gather Sprint D_, 5Patch-For-Review: Performance: onMakeGlobalVariablesScript causes too many internal hits to API - https://phabricator.wikimedia.org/T92580#1115219 (10Jdlrobson) [18:25:20] 10Gather, 3Gather Sprint D_: Collections need a last modified and created field - https://phabricator.wikimedia.org/T92690#1122338 (10Jdlrobson) [18:26:49] Deskana, hi [18:26:55] Krenair: Hello! [18:27:03] Do you know what to do with https://ticket.wikimedia.org/otrs/index.pl?Action=AgentTicketZoom&TicketID=8194589 ? [18:27:11] not sure if it's a partners thing or... [18:27:58] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App, 6Analytics-Engineering: Add page_id and namespace to X-Analytics header in App / api requests - https://phabricator.wikimedia.org/T92875#1122344 (10Ottomata) 3NEW [18:28:23] Krenair: Yeah, that should be handled by the Partners Team. Forward it to Carolynne and she can find the right person/list. [18:28:39] Krenair: cschloeder@wikimedia.org [18:28:45] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App, 6Analytics-Engineering: Add page_id and namespace to X-Analytics header in App / api requests - https://phabricator.wikimedia.org/T92875#1122351 (10Ottomata) [18:30:44] done, ty [18:32:02] 10Gather, 3Gather Sprint C_, 5Patch-For-Review: Adding to new collection workflow - https://phabricator.wikimedia.org/T91296#1122369 (10JKatzWMF) [18:32:03] 3Gather Sprint C_: overlay scroll stops before bottom - https://phabricator.wikimedia.org/T92791#1122370 (10JKatzWMF) [18:32:05] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App, 6Analytics-Engineering: Add page_id and namespace to X-Analytics header in App / api requests - https://phabricator.wikimedia.org/T92875#1122371 (10dr0ptp4kt) I prefer header enrichment be done at the MediaWiki API tier if possible. The parameter name-val... [18:32:56] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App, 6Analytics-Engineering, 10MediaWiki-API: Add page_id and namespace to X-Analytics header in App / api requests - https://phabricator.wikimedia.org/T92875#1122374 (10Krenair) [18:34:19] 6Mobile-Web: No entry point for commons category - https://phabricator.wikimedia.org/T92687#1122383 (10Florian) a:3Florian [18:34:46] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App, 6Analytics-Engineering, 10MediaWiki-API: Add page_id and namespace to X-Analytics header in App / api requests - https://phabricator.wikimedia.org/T92875#1122384 (10yuvipanda) +1 for doing it at the mediawiki API level. I think that would necessitate a... [18:41:47] dr0ptp4kt: FYI - I forgot to say this, but your invite to hockey worked for me [18:43:33] coreyfloyd: good. and i infer from bgerstle that it worked for him, too [18:43:41] yep [18:43:55] the problem before is that i had a hockeyapp account still connected to my spotify email [18:43:57] so i wasn't getting the invites [18:44:36] joakino: rmoen http://hatjitsu.wmflabs.org/19447 [18:44:50] & https://phabricator.wikimedia.org/tag/gather_sprint_d/board/ [18:50:54] (03PS60) 10MaxSem: Recording user responses [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/177448 [18:51:18] 10Gather, 3Gather Sprint D_: Collections need a last modified and created field - https://phabricator.wikimedia.org/T92690#1122476 (10Jdlrobson) [18:52:59] (03PS1) 10Florianschmidtwelzow: Infobox: Add link to commons category overlay [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/197110 (https://phabricator.wikimedia.org/T92687) [18:54:14] (03CR) 10Florianschmidtwelzow: Infobox: Add link to commons category overlay (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/197110 (https://phabricator.wikimedia.org/T92687) (owner: 10Florianschmidtwelzow) [18:55:10] 10Gather, 3Gather Sprint D_, 10Security-Reviews: Security review of Gather extension - https://phabricator.wikimedia.org/T91778#1122498 (10Jdlrobson) [18:57:05] 3Gather Sprint D_: project-wide list of collections - https://phabricator.wikimedia.org/T92782#1122504 (10Jdlrobson) [18:58:09] 10Gather, 3Gather Sprint D_: project-wide list of collections - https://phabricator.wikimedia.org/T92782#1120229 (10Jdlrobson) [18:59:01] 6Mobile-Web: [Tracking] Features in alpha - https://phabricator.wikimedia.org/T86964#1122507 (10Florian) [18:59:56] 10Gather, 3Gather Sprint D_: project-wide list of collections - https://phabricator.wikimedia.org/T92782#1122510 (10Jdlrobson) [19:01:27] 10Gather, 3Gather Sprint D_: project-wide list of collections - https://phabricator.wikimedia.org/T92782#1120229 (10Jdlrobson) [19:04:09] 10Gather, 3Gather Sprint D_: An admin is able to hide or unhide a list. - https://phabricator.wikimedia.org/T91444#1122516 (10Jdlrobson) @yurik can we make it possible for an admin to edit collection info (specifically private/public status)? [19:05:19] 10Gather, 3Gather Sprint D_: An admin is able to hide or unhide a list. - https://phabricator.wikimedia.org/T91444#1122518 (10Jdlrobson) [19:06:01] 10Gather, 3Gather Sprint D_: Article reader sees message/guidance about creating a collection so that they know the feature exists and the entry point - https://phabricator.wikimedia.org/T91015#1122520 (10Jdlrobson) [19:09:11] 10Gather, 3Gather Sprint D_: Article reader sees message/guidance about creating a collection so that they know the feature exists and the entry point - https://phabricator.wikimedia.org/T91015#1122525 (10Jdlrobson) Take a look at PageActionOverlay in mobile. [19:09:18] 10Gather, 3Gather Sprint D_: Article reader sees message/guidance about creating a collection so that they know the feature exists and the entry point - https://phabricator.wikimedia.org/T91015#1122526 (10Jdlrobson) [19:13:11] 10Gather, 3Gather Sprint D_: Event logging for Gather onboarding - https://phabricator.wikimedia.org/T92742#1122546 (10Jdlrobson) [19:15:14] 10Gather, 3Gather Sprint D_: Populate gather with proper feature name - https://phabricator.wikimedia.org/T92665#1122560 (10Jdlrobson) [19:17:07] 10Gather, 3Gather Sprint D_: Event logging for hamburger "collections" link - https://phabricator.wikimedia.org/T92855#1122573 (10Jdlrobson) [19:17:23] 10Gather, 3Gather Sprint D_: Article Deletion handling - https://phabricator.wikimedia.org/T90992#1122576 (10JKatzWMF) a:5JKatzWMF>3None [19:17:44] 10Gather, 3Gather Sprint D_: Event logging for hamburger "collections" link - https://phabricator.wikimedia.org/T92855#1122056 (10Jdlrobson) [19:18:22] 10Gather, 3Gather Sprint D_: Article Deletion handling - https://phabricator.wikimedia.org/T90992#1072323 (10JKatzWMF) [19:18:36] https://etherpad.wikimedia.org/p/Q [19:19:24] 10Gather, 3Gather Sprint D_: Article Deletion handling - https://phabricator.wikimedia.org/T90992#1122586 (10Jdlrobson) [19:20:43] 10Gather, 3Gather Sprint D_: Article Deletion handling - https://phabricator.wikimedia.org/T90992#1122593 (10JKatzWMF) [19:21:03] 10Wikipedia-App-iOS-App: 4.0.7.5 previous edits display their time stamp +7hours - https://phabricator.wikimedia.org/T92696#1122596 (10brion) Ah it's probably showing UTC instead of local time. Should make sure dates are properly formatted with local timezone... [19:23:27] 10Gather, 3Gather Sprint D_, 10Wikimedia-Extension-setup, 3Roadmap, 7user-notice: Enable Extension:Gather (article lists) on English Wikipedia Mobile Beta - https://phabricator.wikimedia.org/T91341#1122604 (10Jdlrobson) [19:23:34] okie poke [19:23:43] gonna see if george is still awake [19:23:53] :/ [19:23:54] 10Gather: Add members to a collection using a search interface - https://phabricator.wikimedia.org/T90991#1122605 (10Jdlrobson) [19:24:58] 10Gather, 3Gather Sprint D_: API unit tests - https://phabricator.wikimedia.org/T92830#1122606 (10Jdlrobson) [19:25:22] kaldari & phuedx, please have your say about storage in 1h or forever remain silent:P [19:25:56] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Configure & document iOS build hardware - https://phabricator.wikimedia.org/T92880#1122608 (10BGerstle-WMF) 3NEW [19:26:10] 10Gather, 3Gather Sprint D_: Move Gather to beta mode - https://phabricator.wikimedia.org/T92881#1122614 (10Jdlrobson) 3NEW [19:26:30] 10Gather, 3Gather Sprint D_: Move Gather to beta mode - https://phabricator.wikimedia.org/T92881#1122623 (10Jdlrobson) [19:28:34] https://phabricator.wikimedia.org/tag/gather_sprint_Diplodocus/ [19:29:27] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Configure Jenkins for iOS builds - https://phabricator.wikimedia.org/T92882#1122640 (10BGerstle-WMF) 3NEW [19:30:24] 10Wikipedia-App-iOS-App, 10Analytics-EventLogging, 6Analytics-Kanban, 3Mobile App Sprint 52 - iOS: Some events not validating for MobileWikiAppSavedPages schema - https://phabricator.wikimedia.org/T91290#1122650 (10Nuria) Talked to Adam and changes here are done, scheduled to be released on march 30th. [19:33:09] 10Gather, 3Gather Sprint Diplodocus: Design tweaks for Vector - https://phabricator.wikimedia.org/T92883#1122656 (10Jdlrobson) 3NEW [19:33:11] http://hatjitsu.wmflabs.org/07516 [19:33:29] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Create a fastlane configuration file for iOS builds - https://phabricator.wikimedia.org/T92884#1122670 (10BGerstle-WMF) 3NEW [19:33:34] 10Gather, 3Gather Sprint Diplodocus: Design tweaks for Vector skin - https://phabricator.wikimedia.org/T92883#1122676 (10Jdlrobson) [19:33:44] http://hatjitsu.wmflabs.org/07516 [19:34:25] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Setup iOS build server and distribute alpha builds internally - https://phabricator.wikimedia.org/T91786#1122679 (10BGerstle-WMF) [19:34:45] https://phabricator.wikimedia.org/project/view/1112/ [19:35:21] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App, 6Analytics-Engineering, 10MediaWiki-API: Add page_id and namespace to X-Analytics header in App / api requests - https://phabricator.wikimedia.org/T92875#1122692 (10Anomie) Hmm, no #MediaWiki-extensions-XAnalytics ? As far as I understand this, that's w... [19:38:08] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Revamp xcode & makefile for build tasks - https://phabricator.wikimedia.org/T92886#1122697 (10BGerstle-WMF) 3NEW [19:40:14] 10Gather, 3Gather Sprint Diplodocus: API returning watchlist with id other than 0 - https://phabricator.wikimedia.org/T92888#1122721 (10Jdlrobson) 3NEW [19:40:49] 10Gather, 3Gather Sprint C_, 5Patch-For-Review: It should not be possible to access the editing interface for watchlist - https://phabricator.wikimedia.org/T92414#1122729 (10Jdlrobson) 5Open>3Resolved Another issue - see https://phabricator.wikimedia.org/T92888#1122721 [19:41:03] 10Gather, 3Gather Sprint C_, 5Patch-For-Review: It should not be possible to access the editing interface for watchlist - https://phabricator.wikimedia.org/T92414#1122731 (10JKatzWMF) Done--actually caused by some other bug with separate ticket. [19:41:23] 3Gather Sprint C_, 5Patch-For-Review: Edit and delete an existing collection UI - https://phabricator.wikimedia.org/T90990#1122733 (10JKatzWMF) 5Open>3Resolved [19:41:39] 10Gather, 3Gather Sprint Diplodocus: API returning watchlist with id other than 0 - https://phabricator.wikimedia.org/T92888#1122734 (10Jdlrobson) Related to this http://en.m.wikipedia.beta.wmflabs.org/wiki/Special:Gather/Jdlrobson/0 should show my watchlist This started happening after following these steps 1... [19:45:41] 3Gather Sprint C_: overlay scroll stops before bottom - https://phabricator.wikimedia.org/T92791#1122740 (10Jdlrobson) [19:46:29] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Configure & document iOS build hardware - https://phabricator.wikimedia.org/T92880#1122742 (10KLans_WMF) [19:46:35] 3Gather Sprint C_, 5Patch-For-Review: Add scrollable element to collections content overlay - https://phabricator.wikimedia.org/T92050#1122744 (10Jdlrobson) 5Open>3Resolved New bug created in next sprint. This is not done though. [19:47:04] 10Gather, 3Gather Sprint Diplodocus: overlay scroll stops before bottom - https://phabricator.wikimedia.org/T92791#1120383 (10Jdlrobson) [19:47:14] 10Gather, 3Gather Sprint Diplodocus: Fix scrolling behaviour - https://phabricator.wikimedia.org/T92791#1122748 (10Jdlrobson) [19:47:36] 10Gather, 7Jenkins: Get qunit setup on Jenkins for Extension:Gather - https://phabricator.wikimedia.org/T91708#1122751 (10Jdlrobson) [19:47:40] 10Gather: Use extension.json, deprecate use of Gather.php for registering config - https://phabricator.wikimedia.org/T91802#1122752 (10Jdlrobson) [19:47:45] 10Gather: Gather should be using a Gruntfile - https://phabricator.wikimedia.org/T92589#1122753 (10Jdlrobson) [19:49:41] 10Gather, 3Gather Sprint C_, 5Patch-For-Review: Adding to new collection workflow - https://phabricator.wikimedia.org/T91296#1122766 (10Jdlrobson) 5Open>3Resolved [19:50:19] (03PS4) 10Jdlrobson: Create a base class for our content overlays [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196495 (https://phabricator.wikimedia.org/T92651) [19:50:40] 10Gather, 3Gather Sprint Diplodocus, 5Patch-For-Review: Deletion overlay broken, prevents editing of collections - https://phabricator.wikimedia.org/T92651#1122770 (10Jdlrobson) [19:50:43] 10Wikipedia-App-iOS-App, 3Mobile App Sprint 52 - iOS: Cut store release with only the addition of HockeyApp - https://phabricator.wikimedia.org/T91094#1122773 (10KLans_WMF) 5Open>3declined [19:50:47] 10Gather, 3Gather Sprint Diplodocus, 5Patch-For-Review: Collections are restricted to 10 items without pagination - https://phabricator.wikimedia.org/T90903#1122774 (10Jdlrobson) [19:51:39] 10Gather, 3Gather Sprint Diplodocus: watchlist overlay: design tweaks - https://phabricator.wikimedia.org/T92037#1122778 (10Jdlrobson) [19:51:42] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: image gallery UI doesn't flip to suit RTL languages - https://phabricator.wikimedia.org/T91313#1122779 (10KLans_WMF) [19:52:19] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Revamp xcode & makefile for build tasks - https://phabricator.wikimedia.org/T92886#1122789 (10KLans_WMF) [19:52:46] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: image gallery UI doesn't flip to suit RTL languages - https://phabricator.wikimedia.org/T91313#1122794 (10Deskana) a:3BGerstle-WMF [19:53:20] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: image gallery UI doesn't flip to suit RTL languages - https://phabricator.wikimedia.org/T91313#1079644 (10Deskana) @BGerstle-WMF said he was handling this one, so I assigned it to him. He can correct this if I'm wrong. [19:53:29] 10Gather, 3Gather Sprint Diplodocus: Caching issues? - https://phabricator.wikimedia.org/T92573#1122797 (10Jdlrobson) [19:53:59] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Some map images cause the entire webview to be scrollable left to right - https://phabricator.wikimedia.org/T91702#1122803 (10KLans_WMF) [19:54:21] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Handle nested parentheses in Share a Fact even better - https://phabricator.wikimedia.org/T91846#1122805 (10KLans_WMF) [19:55:22] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: 4.0.6.3 Image Gallery - some images displayed in low resolution - https://phabricator.wikimedia.org/T91124#1122815 (10KLans_WMF) [19:55:28] MaxSem: i am happy to see it move forward \o/ [19:55:38] +1? [19:57:16] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Create a fastlane configuration file for iOS builds - https://phabricator.wikimedia.org/T92884#1122819 (10KLans_WMF) [19:59:16] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Configure Jenkins for iOS builds - https://phabricator.wikimedia.org/T92882#1122843 (10BGerstle-WMF) [20:00:26] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: Configure Jenkins for iOS builds - https://phabricator.wikimedia.org/T92882#1122846 (10KLans_WMF) [20:00:59] (03CR) 10Phuedx: [C: 04-1] "Two MobileFrontend's snuck through." (032 comments) [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/197092 (owner: 10Bmansurov) [20:02:35] (03PS2) 10Bmansurov: Add dev-scripts for easy development [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/197092 [20:04:06] bgerstle: coreyfloyd i have a couple small patches for cr if you have a few minutes... [20:04:31] https://gerrit.wikimedia.org/r/197005 & https://gerrit.wikimedia.org/r/197006 [20:21:14] mhurd k, will have a look in a bit [20:21:21] bgerstle: no rush [20:24:50] kristenlans: hey, did you want to talk about any of the tasks? sorry that meeting ended a little abruptly [20:25:51] Maryana, the audio wasn't that good - so when's the last day and when are the drinks? [20:26:09] tbd :) [20:26:23] even the last day? o_0 [20:27:25] kaldari, please have your say about storage in 1h or forever remain silent:P [20:29:13] MaxSem: ok... [20:31:25] bgerstle: thanks for checking in on that :-) I was actually going to send an email to "wrap up" (note our velocity/poitns planned, ask for commitment). Otherwise I think it's looking pretty solid. [20:31:56] (03CR) 10Kaldari: [C: 04-1] Recording user responses (031 comment) [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/177448 (owner: 10MaxSem) [20:32:22] ok, thanks [20:32:27] lol, i missed it:P [20:32:54] mhurd: coreyfloyd need to run and pick up the car from the shop. bbiab [20:33:04] bgerstle: see ya! [20:33:23] (03PS61) 10MaxSem: Recording user responses [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/177448 [20:33:27] nuria: I need to get a dump of the existing WikiGrok data from the eventlogging table (log.MobileWebWikiGrokResponse_10352279), but I seem to have permission trouble whenever I try to dump it. Do you have time to help? [20:33:31] kaldari, phuedx ^^ [20:35:35] (03CR) 10Deskana: [C: 031] "WFM." [apps/android/wikipedia] - 10https://gerrit.wikimedia.org/r/197098 (owner: 10BearND) [20:36:23] 10Gather, 3Gather Sprint Diplodocus: API unit tests - https://phabricator.wikimedia.org/T92830#1122979 (10Yurik) [20:36:24] 3Gather Sprint Diplodocus: Api bug when loading action=query&list=lists&lstids=X and logged in - https://phabricator.wikimedia.org/T92865#1122978 (10Yurik) [20:36:37] 10Gather, 3Gather Sprint Diplodocus: API unit tests - https://phabricator.wikimedia.org/T92830#1121379 (10Yurik) [20:36:38] 10Gather, 3Gather Sprint Diplodocus: API returning watchlist with id other than 0 - https://phabricator.wikimedia.org/T92888#1122985 (10Yurik) [20:41:44] 10MobileFrontend-Feature-requests, 10Vector: Switch to MobileFrontend when viewport is narrower than 694px - https://phabricator.wikimedia.org/T92894#1122997 (10Jaredzimmerman-WMF) 3NEW [20:43:35] (03CR) 10Jdlrobson: [C: 04-1] Infobox: Add link to commons category overlay (032 comments) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/197110 (https://phabricator.wikimedia.org/T92687) (owner: 10Florianschmidtwelzow) [20:44:03] rmoen: board is up and running [20:44:08] jdlrobson: the first two cards in ready for dev are backend heavy, do you mind if i take https://phabricator.wikimedia.org/T92782 / [20:44:09] ? [20:44:18] rmoen: go for it - take what you can get done quickest [20:44:21] we need to clear entire board :) [20:44:38] rmoen: although that is blocked on api changes.. i think? [20:44:45] you can fake it in mean time i guess? [20:45:20] jdlrobson, won't fix any bugs until unit tests - too shaky otherwise [20:45:23] jdlrobson: yeah its blocked as currently you have to specify a user to get lists for [20:46:23] nuria: nevermind, got it figured out [20:46:31] jdlrobson: also IMO pagination for that special page probably needs to be pulled out of that card [20:48:02] jdlrobson: rmoen MaxSem: When you build your Gather API would you mind building it so that it's easy to add a couple of flags to each list? For apps, I'd like to have "make available for offline" and "private" flags if you don't already have them. [20:48:23] yurik, ^ [20:48:55] bearND, it is currently storing data as blob [20:49:03] so extra props are easy [20:49:10] adding something indexable is a different matter [20:50:15] create table gather_props(gp_list int, gp_name varchar(60), gp_value blob) ? [20:51:21] mhurd: just bugging out for an hour. Will look when I get back. [20:51:29] coreyfloyd: sounds good! [20:51:48] yurik: i guess that's good enough for now but likely to change I assume [20:52:11] 10Gather, 3Gather Sprint Diplodocus: Article reader sees message/guidance about creating a collection so that they know the feature exists and the entry point - https://phabricator.wikimedia.org/T91015#1123060 (10rmoen) a:5MSyed>3rmoen [20:52:30] bearND, btw, private is already in. Can easily add "offline" [20:52:36] kaldari: ok [20:52:44] yurik: great [20:53:04] kaldari: let me know if you need anything else [20:53:12] bearND, phabricator task pls, and mark it blocked on API tests [20:55:50] yurik: well, we don't need it right now but will probably need it later once apps are doing something in that regard, so I'm hesitant to add a Phab task just yet [20:56:01] oki [21:04:15] 6Mobile-Web: Tappable area of close button on reference drawer is too small - https://phabricator.wikimedia.org/T92700#1123116 (10bmansurov) a:3bmansurov [21:06:40] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App, 6Analytics-Engineering, 10MediaWiki-API: Add page_id and namespace to X-Analytics header in App / api requests - https://phabricator.wikimedia.org/T92875#1123130 (10Deskana) Seems like it needs to be done in the extension, rather than in the app. [21:08:22] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App: Portal template displays an additional element instead of being rendered as just links - https://phabricator.wikimedia.org/T92659#1123142 (10Deskana) p:5Triage>3Low [21:11:03] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App: Image map: set a slight delay and outline the area before redirecting - https://phabricator.wikimedia.org/T92426#1123177 (10Deskana) p:5Triage>3Low [21:12:00] 10Wikipedia-App-Android-App: Need fade in fade out for action bar in a few interactions - https://phabricator.wikimedia.org/T92314#1123187 (10Deskana) a:5Dbrant>3None [21:12:00] (03PS2) 10Jdlrobson: POC/WIP: Add Special:GatherEditFeed [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196732 [21:12:07] 10Wikipedia-App-Android-App: Need fade in fade out for action bar in a few interactions - https://phabricator.wikimedia.org/T92314#1106116 (10Deskana) p:5Triage>3Normal [21:12:26] 10Gather, 3Gather Sprint Diplodocus: When I go to Special:Gather and see the collections I own they are illustrated with a page image - https://phabricator.wikimedia.org/T91675#1123191 (10Jdlrobson) a:3Jdlrobson [21:13:14] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App: Category view shows no articles in the mobile app - https://phabricator.wikimedia.org/T71984#1123202 (10Deskana) p:5Triage>3Low [21:13:19] (03CR) 10Jdlrobson: [C: 032] "feel free to make voting whenever necessary. Thanks lego!" [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196972 (https://phabricator.wikimedia.org/T90943) (owner: 10Legoktm) [21:14:30] can anyone send me a link to the non-js search page on mobile? [21:14:38] 10Gather, 3Gather Sprint Diplodocus, 5Patch-For-Review: Collections are restricted to 10 items without pagination - https://phabricator.wikimedia.org/T90903#1123208 (10Jdlrobson) 5Open>3Resolved [21:14:43] (03Merged) 10jenkins-bot: Add "composer test" command to run phpcs, and fix warnings [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196972 (https://phabricator.wikimedia.org/T90943) (owner: 10Legoktm) [21:15:43] (03PS5) 10Robmoen: Create a base class for our content overlays [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196495 (https://phabricator.wikimedia.org/T92651) (owner: 10Jdlrobson) [21:16:43] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App: Improving the Gallery UX - https://phabricator.wikimedia.org/T91329#1123218 (10Deskana) p:5Triage>3Normal a:5Deskana>3None [21:18:27] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App, 6Zero, 7Design: only show features related to Zero if Zero is relevant in this context - https://phabricator.wikimedia.org/T92267#1123235 (10Deskana) a:5Deskana>3None [21:18:53] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App: Allow saved pages in mobile apps to be organized - https://phabricator.wikimedia.org/T92255#1123237 (10Deskana) [21:19:15] (03PS2) 10Jdlrobson: Hygiene: Use one api request to access meta data and members for viewing collection [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196641 [21:19:43] kaldari: got a sec to talk about icon sizes? [21:19:53] sure [21:20:17] kaldari: currently, icon widths in beta and alpha are 54.4px, not 44px [21:20:17] 10Gather, 3Gather Sprint Diplodocus, 5Patch-For-Review: Performance: onMakeGlobalVariablesScript causes too many internal hits to API - https://phabricator.wikimedia.org/T92580#1123250 (10Jdlrobson) a:3Jdlrobson [21:20:31] kaldari: are we going to make them smaller too? [21:20:36] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App: Allow saved pages in mobile apps to be organized - https://phabricator.wikimedia.org/T92255#1104153 (10Deskana) 5Open>3stalled This overlaps significantly with #Gather, and is probably something that we'll tackle if/when Gather makes its way to the mobi... [21:20:52] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App: Allow saved pages in mobile apps to be organized - https://phabricator.wikimedia.org/T92255#1123254 (10Deskana) p:5Triage>3Normal [21:21:14] bmansurov: no, we can leave them larger for now. We just need to make sure that any icon buttons are at least 44x44px. [21:21:27] ok, at least ;) [21:21:29] thanks [21:21:37] yeah, bigger is fine, but not smaller [21:21:42] got it [21:21:48] per the design team [21:21:56] ok [21:22:03] 10Wikipedia-App-Android-App, 10Wikimedia-Hackathon-2015: Page footer doesn't show on short articles where the footer should be displayed above the fold - https://phabricator.wikimedia.org/T88509#1123260 (10Deskana) [21:22:10] 10Wikipedia-App-Android-App, 10Wikimedia-Hackathon-2015: Page footer doesn't show on short articles where the footer should be displayed above the fold - https://phabricator.wikimedia.org/T88509#1013668 (10Deskana) I may try and work on this at the Hackathon. [21:22:13] dbrant: https://phabricator.wikimedia.org/T88509 [21:22:19] bmansurov: BTW, you mentioned that your research project was done. I would love to take a look at it [21:22:37] kaldari: https://gerrit.wikimedia.org/r/#/c/195818/ [21:22:45] 10Wikipedia-App-Android-App: Expose search through intent - https://phabricator.wikimedia.org/T88364#1123267 (10Deskana) p:5Triage>3Normal [21:22:54] kaldari: thanks for taking a look [21:23:34] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App, 6Mobile-Apps, 7Tracking: Hackathon-able / volunteer-worthy tasks for Mobile Apps - https://phabricator.wikimedia.org/T91510#1123270 (10Dbrant) [21:23:35] 10Wikipedia-App-Android-App, 10Wikimedia-Hackathon-2015: Page footer doesn't show on short articles where the footer should be displayed above the fold - https://phabricator.wikimedia.org/T88509#1123271 (10Dbrant) [21:24:27] 10Wikipedia-App-Android-App: NPE crash from DateFormat.getRelativeTimeSpanString calls - https://phabricator.wikimedia.org/T88202#1123275 (10Deskana) p:5Low>3Lowest The combination of "Samsung" and "2.3" is particularly lethal to the user's ability to... do anything with their phone. [21:25:24] mhurd coreyfloyd back from the shop [21:26:53] 10Wikipedia-App-Android-App: NPE crash from com.android.internal.policy.impl.PhoneWindow.onKeyUpPanel - https://phabricator.wikimedia.org/T87722#1123287 (10Deskana) 5Open>3declined a:3Deskana This is code outside the app. There's not much we can do. :-( [21:27:34] 10Wikipedia-App-iOS-App: Handle first time launch with case of user credentials already stored - https://phabricator.wikimedia.org/T92732#1123294 (10dr0ptp4kt) [21:28:45] kristenlans: hi [21:29:06] do you prefer I don't open bugs for the alpha version ? [21:29:07] 10Wikipedia-App-Android-App: Page image gallery starts in the middle, difficult to see all images - https://phabricator.wikimedia.org/T87622#1123311 (10Deskana) [21:30:28] 10Wikipedia-App-Android-App: Add image ordering to apps content service, and use it in image gallery, so that images are always in order in the gallery - https://phabricator.wikimedia.org/T86954#1123331 (10Deskana) [21:31:55] (03CR) 10Jdlrobson: [C: 04-1] "Also please add tests." (031 comment) [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196739 (owner: 10Yurik) [21:32:02] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App: Read more autogenerated? suggestions don't make much sense outside of content namespaces - https://phabricator.wikimedia.org/T92787#1123340 (10Deskana) p:5Triage>3Normal [21:32:18] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App: Read more autogenerated? suggestions don't make much sense outside of content namespaces - https://phabricator.wikimedia.org/T92787#1120319 (10Deskana) We should just disable these outside mainspace. [21:33:00] 10Wikipedia-App-iOS-App: Don't display "You can read this article in n other languages" when n = 0 - https://phabricator.wikimedia.org/T92786#1123352 (10Deskana) [21:33:17] 10Wikipedia-App-iOS-App, 7Easy: Don't display "You can read this article in n other languages" when n = 0 - https://phabricator.wikimedia.org/T92786#1120305 (10Deskana) [21:33:36] 10Wikipedia-App-iOS-App, 7Easy: Don't display "You can read this article in n other languages" when n = 0 - https://phabricator.wikimedia.org/T92786#1120305 (10Deskana) @dr0ptp4kt thinks this is #Easy. [21:34:38] 10Wikipedia-App-iOS-App: White article title hard to read when backround image is also white - https://phabricator.wikimedia.org/T92785#1123390 (10Deskana) p:5Triage>3Normal We should make the gradient or shadow on the text more pronounced. [21:36:13] 10Wikipedia-App-iOS-App: White article title hard to read when backround image is also white - https://phabricator.wikimedia.org/T92785#1120284 (10Deskana) @dr0ptp4kt and @dbrant said there are APIs in iOS and Android that can give you the primary colour of a bitmap... one could then take the inverse and use tha... [21:37:09] 10Wikipedia-App-iOS-App: Confusing nested "quick facts" in iOS app - https://phabricator.wikimedia.org/T92784#1123435 (10Deskana) p:5Triage>3Low [21:37:57] 10Wikipedia-App-iOS-App: Handle first time launch with case of user credentials already stored - https://phabricator.wikimedia.org/T92732#1123438 (10Deskana) p:5Triage>3Lowest [21:38:07] (03PS1) 10Bmansurov: Make the tappable area of the reference drawer close button bigger [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/197214 (https://phabricator.wikimedia.org/T92700) [21:40:13] 10Wikipedia-App-iOS-App: Consider disabling Save and Share button when article load in progress - https://phabricator.wikimedia.org/T92731#1119054 (10Deskana) #Design may want to give some input. [21:40:18] kaldari: can you send me a link to the non-js search page on mobile? [21:41:32] dbrant: https://commons.wikimedia.org/wiki/File:Dubna.house_of_scientiests.jpg [21:42:08] kaity: https://en.m.wikipedia.org/wiki/Special:Search [21:42:19] thanks! [21:42:19] it’s not much to look at :) [21:42:41] Matanya that is a question for Maryana :-) [21:42:48] oh do i have to turn off js to see the search field on the page? [21:42:58] kaity: If you’re using Safari, you can turn off Javascript in your prefs [21:43:37] kaity: Yes, the page looks different if you have JS off [21:43:49] 10Wikipedia-App-iOS-App: Wikitext sometimes shows in image viewer - https://phabricator.wikimedia.org/T92729#1123464 (10Deskana) 5Open>3Resolved a:3Deskana This was an error in the description. I fixed it myself. [21:44:02] kaldari: got it, thanks [21:44:17] coreyfloyd: mhurd just about wrapping up the image gallery RTL patch. then will move to the bigger blocker [21:44:27] i had already started, and didn't want to switch tasks (again) [21:44:31] but, i have a ? for you [21:44:37] both* [21:44:59] 10Wikipedia-App-iOS-App: NSURLDomain error sometimes happens on login screen - https://phabricator.wikimedia.org/T92727#1123471 (10Deskana) p:5Triage>3Low [21:45:07] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App: Read more autogenerated? suggestions don't make much sense outside of content namespaces - https://phabricator.wikimedia.org/T92787#1123474 (10Legoktm) Why mainspace as opposed to content namespaces? [21:46:11] 10Wikipedia-App-iOS-App: [4.0.7.5] Lead Image & Image Gallery features broken for migrated articles - https://phabricator.wikimedia.org/T92697#1123478 (10Deskana) p:5Triage>3Normal [21:47:21] 10Wikipedia-App-iOS-App: [4.0.7.5] Lead Image & Image Gallery features broken for migrated articles - https://phabricator.wikimedia.org/T92697#1118359 (10Deskana) p:5Normal>3Low It's not the biggest deal that saved pages are not showing lead images if they're migrated, because they didn't have lead images be... [21:48:09] ohhh boy [21:48:22] 10Wikipedia-App-iOS-App: 4.0.7.5 previous edits display their time stamp +7hours - https://phabricator.wikimedia.org/T92696#1123484 (10Deskana) p:5Triage>3Normal [21:48:23] mhurd coreyfloyd are you seeing a crash on startup? [21:48:45] (03PS4) 10Jdlrobson: Ability to see or filter on lists containing title [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196739 (owner: 10Yurik) [21:48:47] (03PS1) 10Jdlrobson: Don't check watch status on special pages [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197221 (https://phabricator.wikimedia.org/T92580) [21:48:49] (03PS1) 10Jdlrobson: Reduce API requests on any given non-special page to 1 [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197222 (https://phabricator.wikimedia.org/T92580) [21:48:58] 10Wikipedia-App-iOS-App: Use system dictionary suggestion for Did You Mean? when server response is insufficient - https://phabricator.wikimedia.org/T92658#1123496 (10Deskana) p:5Triage>3Normal [21:49:03] shite yurik is gonna kill me. Didn't mean to do that :-S [21:49:18] (03PS5) 10Jdlrobson: Ability to see or filter on lists containing title [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196739 (owner: 10Yurik) [21:49:31] (03CR) 10Jdlrobson: "My bad. Sorry, didn't mean to upload new patchset." [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196739 (owner: 10Yurik) [21:49:33] 10Wikipedia-App-iOS-App: [4.0.7.5] Lead Image & Image Gallery features broken for migrated articles - https://phabricator.wikimedia.org/T92697#1123502 (10BGerstle-WMF) @Deskana haven't started working on it yet. Was planning on picking it up soon, though. [21:49:40] jdlrobson, !!!! [21:49:53] dr0ptp4kt: seeing a crash on startup in iOS 6 [21:49:55] sorryyyyyyyy i didn't touch your patch i swear [21:49:56] grr, stop uploading patches [21:50:01] )) [21:50:05] rright [21:50:07] i was just trying to do the frontend change on top of the api change [21:50:13] failing to parse this timestamp: @"2015-02-10T10:31:27Z" [21:50:31] trying again on sim [21:50:37] (and yes, on a clean install) [21:50:51] (03PS2) 10Jdlrobson: Reduce API requests on any given non-special page to 1 [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197222 (https://phabricator.wikimedia.org/T92580) [21:50:51] 10Wikipedia-App-iOS-App: 4.0.7.4 Main Page article is not scrollable - https://phabricator.wikimedia.org/T92655#1123507 (10Deskana) 5Open>3Invalid a:3Deskana Since it's by design that the main page has less content on mobile, @Etonkovidova said that this is actually invalid. [21:50:52] oh wait [21:50:59] sorry, the device was also set to arabic [21:51:10] 10Wikipedia-App-Localization-or-Translation-or-Language, 7Tracking: Support language variants (LanguageConverter) - https://phabricator.wikimedia.org/T64416#1123513 (10wctaiwan) [21:51:11] 10Wikipedia-App-Android-App, 10Wikipedia-App-Localization-or-Translation-or-Language, 7I18n: Special case Traditional and Simplified Chinese in all language handling - https://phabricator.wikimedia.org/T62743#1123511 (10wctaiwan) 5Invalid>3Open Is this no longer an issue? I'm confused what context this b... [21:52:14] bgerstle: eek, is it ok? [21:52:41] seems OK on iOS 8... [21:53:04] 10Gather: I see a share button at the top of the list - https://phabricator.wikimedia.org/T91002#1123516 (10Jdlrobson) Be sure to flesh out the cards better so the entire team can follow and volunteers can build outside sprints. Cards with no description are not helpful - I suggest putting draft/placeholder in t... [21:53:13] dr0ptp4kt: trying again now [21:54:23] ok, definitely crashing on startup in iOS 6 + arabic. (edge case?) [21:54:34] EN seems fine [21:55:27] (03PS1) 10Legoktm: jenkins job validation, do not submit [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197224 [21:55:37] (03CR) 10Robmoen: [C: 04-1] Create a base class for our content overlays (031 comment) [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196495 (https://phabricator.wikimedia.org/T92651) (owner: 10Jdlrobson) [21:55:43] 10Wikipedia-App-iOS-App: Search in different languages: clearer indication that a user is searching in language specific Wiki? - https://phabricator.wikimedia.org/T92528#1123529 (10Deskana) p:5Triage>3Normal [21:55:56] dbrant: https://phabricator.wikimedia.org/T92528 [21:56:06] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App: Search in different languages: clearer indication that a user is searching in language specific Wiki? - https://phabricator.wikimedia.org/T92528#1113910 (10Deskana) [21:56:44] dr0ptp4kt: the solution is to explicitly set the locale on the date formatter [21:56:49] i'm doing it locally now so i can test RTL stuff [21:56:53] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App, 6Mobile-Apps, 7Tracking: Hackathon-able / volunteer-worthy tasks for Mobile Apps - https://phabricator.wikimedia.org/T91510#1123540 (10Dbrant) [21:56:54] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App: Search in different languages: clearer indication that a user is searching in language specific Wiki? - https://phabricator.wikimedia.org/T92528#1123541 (10Dbrant) [21:56:59] (03Abandoned) 10Legoktm: jenkins job validation, do not submit [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197224 (owner: 10Legoktm) [21:57:18] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App: Clear cached search results when user switches language - https://phabricator.wikimedia.org/T92908#1123543 (10Deskana) 3NEW [21:57:24] (03CR) 10Legoktm: "Awesome, phpcs is now voting!" [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196972 (https://phabricator.wikimedia.org/T90943) (owner: 10Legoktm) [21:58:17] 10Wikipedia-App-iOS-App: App MediaViewer jarringly loads cropped low quality image and then higher quality full size image - https://phabricator.wikimedia.org/T92446#1123561 (10Deskana) p:5Triage>3High This isn't a release blocker necessarily, but should be fixed ASAP. [21:58:34] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: App MediaViewer jarringly loads cropped low quality image and then higher quality full size image - https://phabricator.wikimedia.org/T92446#1111014 (10Deskana) [21:59:03] bgerstle: cool. coreyfloyd, is the thing bgerstle describes, and https://phabricator.wikimedia.org/T92696, related? [21:59:46] (03PS6) 10Jdlrobson: Create a base class for our content overlays [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196495 (https://phabricator.wikimedia.org/T92651) [21:59:47] 10Wikipedia-App-iOS-App: Image map: better handling of image info? - https://phabricator.wikimedia.org/T92423#1123568 (10Deskana) p:5Triage>3Low [22:01:10] (03CR) 10Robmoen: [C: 032] Create a base class for our content overlays [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196495 (https://phabricator.wikimedia.org/T92651) (owner: 10Jdlrobson) [22:02:05] (03CR) 10Jdlrobson: "eek i think we are duplicating work see https://gerrit.wikimedia.org/r/#/c/196641/2" [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197085 (https://phabricator.wikimedia.org/T92862) (owner: 10Jhernandez) [22:02:31] (03PS3) 10Jdlrobson: Hygiene: Use one api request to access meta data and members for viewing collection [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196641 (https://phabricator.wikimedia.org/T92580) [22:02:33] (03Merged) 10jenkins-bot: Create a base class for our content overlays [extensions/Gather] - 10https://gerrit.wikimedia.org/r/196495 (https://phabricator.wikimedia.org/T92651) (owner: 10Jdlrobson) [22:03:07] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App: Handle edit conflicts on the mobile app in some way - https://phabricator.wikimedia.org/T66281#1123583 (10Krenair) How is this just normal priority? [22:03:50] dr0ptp4kt: looking [22:12:08] dr0ptp4kt: that doesn't seem related. [22:12:54] 10Wikipedia-App-Android-App, 10Wikipedia-App-iOS-App: Handle edit conflicts on the mobile app in some way - https://phabricator.wikimedia.org/T66281#1123593 (10Deskana) @Krenair We're focussing on reading and not actively supporting editing right now, and also this is a nontrivial issue to fix. [22:14:33] MaxSem: hey max i just noticed some of our nearby querys, today for the first time, don’t have “coordinates” returned with results sometimes. did something change recently with how nearby queries are handled? [22:14:59] coreyfloyd: cool [22:15:41] mmm, ES update lag? can't say anything without examples [22:16:27] dr0ptp4kt: they may be tangentially related. As in they both deal with dates, but the latter bug is probably more like the dates are created without specifying the time zone. [22:19:18] MaxSem: https://en.m.wikipedia.org/w/api.php?action=query&codistancefrompoint=37.332331%7C-122.031219&colimit=50&format=jsonfm&generator=geosearch&ggscoord=37.332331%7C-122.031219&ggslimit=50&ggsradius=10000&pilimit=50&pithumbsize=144&prop=coordinates%7Cpageimages%7Cpageterms&wbptterms=description [22:19:48] MaxSem: the “Santa Clara, California” result is missing coordinates [22:20:09] MaxSem: as is “Santa Clara County, California" [22:24:06] MaxSem: i think my preference for a geosearch query would be it never return a result for which it has no geo coords :) [22:25:09] geosearch and coordinates are coming from different sources, validating against each other would be pita [22:27:54] (03CR) 10Bmansurov: Hygiene: Use same saving indication in TalkSectionAdd like in EditorOverlay (031 comment) [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/193522 (https://phabricator.wikimedia.org/T91139) (owner: 10Florianschmidtwelzow) [22:28:33] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: App MediaViewer jarringly loads cropped low quality image and then higher quality full size image - https://phabricator.wikimedia.org/T92446#1123626 (10BGerstle-WMF) @Deskana I'll ping you & @Vibhabamba when I pick this up. In the meantime, could you both so... [22:29:35] 10Wikipedia-App-Android-App, 3Mobile-App-Sprint-53-Android: BLOCKED: As an engaged reader, I'd like to know that I can share cards with interesting facts on. - https://phabricator.wikimedia.org/T89988#1123628 (10KLans_WMF) [22:29:45] (03CR) 10Bmansurov: [C: 031] Don't check watch status on special pages [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197221 (https://phabricator.wikimedia.org/T92580) (owner: 10Jdlrobson) [22:29:48] 10Wikipedia-App-Android-App: BLOCKED: As an engaged reader, I'd like to know that I can share cards with interesting facts on. - https://phabricator.wikimedia.org/T89988#1050896 (10KLans_WMF) [22:30:35] mhurd it's kinda sad that the official Yosemite Nat'l Park article has an OSX disambiguation blurb :-P [22:30:37] 10Wikipedia-App-Android-App: BLOCKED: As an engaged reader, I'd like to know that I can share cards with interesting facts on. - https://phabricator.wikimedia.org/T89988#1050896 (10KLans_WMF) Note that we are backlogging this task, as we are blocked on design. [22:30:48] bgerstle: haha [22:32:01] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: App MediaViewer jarringly loads cropped low quality image and then higher quality full size image - https://phabricator.wikimedia.org/T92446#1123642 (10Deskana) The most jarring thing about this is that the image resizes, mostly up (but sometimes down!), when... [22:35:16] 10Wikipedia-App-Android-App, 3Mobile-App-Sprint-53-Android: As a product owner, I'd like to know how well the Tweet A Fact onboarding did at exposing users to the feature - https://phabricator.wikimedia.org/T89989#1050905 (10KLans_WMF) Note that we are backlogging this task, as we are blocked on design. [22:35:29] 10Wikipedia-App-Android-App, 3Mobile-App-Sprint-53-Android: Epic: Onboarding, both generally and for Tweet A Fact - https://phabricator.wikimedia.org/T89985#1050863 (10KLans_WMF) Note that we are backlogging this task, as we are blocked on design. [22:37:17] 10Wikipedia-App-Android-App: As a product owner, I'd like to know how well the Tweet A Fact onboarding did at exposing users to the feature - https://phabricator.wikimedia.org/T89989#1123667 (10KLans_WMF) [22:37:45] 10Wikipedia-App-Android-App: Epic: Onboarding, both generally and for Tweet A Fact - https://phabricator.wikimedia.org/T89985#1050863 (10KLans_WMF) [22:37:56] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: App MediaViewer jarringly loads cropped low quality image and then higher quality full size image - https://phabricator.wikimedia.org/T92446#1123669 (10BGerstle-WMF) Eck, sorry for the funky indentation in my comment *shakes fist angrily at phabricator*. >>!... [22:39:47] bgerstle: mhurd coreyfloyd, would setting xcode 6.3 as the minimum require that a volunteer be part of the developer program to get 6.3 (in beta)? or is there a way for the volunteer to get by with 6.2? [22:40:04] dr0ptp4kt: is it in beta still? i thought it was offiical [22:40:16] damn [22:40:56] dammmniiiit [22:41:04] guess we'll have to wait [22:41:15] or be super cool bleeding edge [22:41:29] dr0ptp4kt: sorry i didn't realize that, guess we'll have to settle for 6.2 for now [22:41:35] bmansurov: what’s your wikidata user name (official WMF account)? [22:41:45] Deskana: dr0ptp4kt do we call this "done"? https://phabricator.wikimedia.org/T90255 [22:41:50] kaldari: let me check [22:42:56] mhurd dr0ptp4kt coreyfloyd stepping out for dog walk, be on later to wrap up RTL. works on iOS 6 & 8, just need to reverse swiping direction [22:42:57] Deskana dr0ptp4kt also this guy: https://phabricator.wikimedia.org/T86692 [22:43:33] laterz o/ [22:43:56] kaldari: Bmansurov (WMF) [22:44:44] kristenlans: Deskana i think https://phabricator.wikimedia.org/T90255 can be closed. we know that https://phabricator.wikimedia.org/T92031 is the thing that would yield the greatest gain (android is proof), and that's tracked separately. [22:47:15] kristenlans: Deskana to the extent that https://phabricator.wikimedia.org/T86692 tracks engineering work, that is 'Done' for iOS. we may resubmit, but that seems besides the point. [22:48:03] dr0ptp4kt: Sounds fine. I think there's still a lot of room for performance work, but the scope was to get it "good enough for release" which I think it is. [22:48:18] thx dr0ptp4kt Deskana one last one: https://phabricator.wikimedia.org/T90981 [22:48:43] kristenlans: carry that to the current sprint cc Deskana [22:48:50] sprint 53 [22:48:53] please [22:49:03] 10Wikipedia-App-iOS-App, 3Mobile-App-Sprint-53-iOS: EPIC: Setup Xcode Server - https://phabricator.wikimedia.org/T90981#1123685 (10KLans_WMF) [22:49:15] thx kristenlans [22:49:58] 10Wikipedia-App-iOS-App, 3Mobile App Sprint 52 - iOS: EPIC: Hone in on, and eradicate, article load time performance issues - https://phabricator.wikimedia.org/T90255#1123688 (10Deskana) 5Open>3Resolved a:3Deskana The scope of this task was to get performance good enough for a release, which it probably... [22:55:04] Deskana: want to work on those ios app screenshots tomorrow? one can only update screenshots as part of a new submission or a not-yet-approved submission. [22:55:22] Deskana: wednesday would also be okay if that's the "deadline" for any final submission [22:55:31] (as opposed to tomorrow) [22:55:38] dr0ptp4kt: Wednesday would be a lot better, my calendar isn't nice tomorrow. [22:55:47] Deskana: i should looked at that first. ok [22:56:58] (03PS1) 10Mhurd: Fix so status bar tap causes web view to scroll to top on iOS 6 and 7. [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/197244 [22:58:41] 10Gather: Only call API dynamically except for special:gather - https://phabricator.wikimedia.org/T92711#1123707 (10Jdlrobson) I understand Gather is making too many queries right now. This is something being worked on. The three patches on T92580 should reduce things considerably. My question is what is the di... [23:02:20] 10Gather, 3Gather Sprint Diplodocus: Only call API dynamically except for special:gather - https://phabricator.wikimedia.org/T92711#1123721 (10Jdlrobson) [23:08:27] coreyfloyd: dr0ptp4kt what's the dealio with https://phabricator.wikimedia.org/T91788, https://phabricator.wikimedia.org/T91790, and https://phabricator.wikimedia.org/T91791? [23:11:59] (03PS3) 10MaxSem: WIP: aggregation [extensions/WikiGrok] - 10https://gerrit.wikimedia.org/r/196136 [23:12:10] weee ^^^ [23:17:40] kristenlans: just acknowledging that i saw your message. i'm looking at the cards. are you wondering if those should be in sprint 53 by any chance? [23:17:58] kristenlans: also, it's getting very late for you [23:18:24] dr0ptp4kt: yes, I'm wondering if they shoudl actually be in the sprint. [23:18:28] 10Wikipedia-App-Android-App: Scrolling in description area isn't easily possible when a link takes too much place - https://phabricator.wikimedia.org/T87638#1123749 (10Etonkovidova) Tried it on Nexus 5 Android 4.4.4 - the vertical scroll feels controllable enough. [23:18:34] dr0ptp4kt: almost tiem fro dinner and Vikings! [23:18:57] lagertha [23:19:08] will [23:19:09] Lagertha <3 [23:19:13] bring [23:19:14] the [23:19:15] noise [23:19:19] lololol [23:20:28] kristenlans: i think https://phabricator.wikimedia.org/T91791 should go into the tech backlog [23:20:46] ^ coreyfloyd [23:21:14] Headed to the dentist... again. [23:21:19] Be back soon, hopefully. [23:21:26] aww Deskana good luck with your grill [23:22:11] kristenlans: i also think https://phabricator.wikimedia.org/T91790 should go to the tech backlog [23:22:15] cc coreyfloyd ^ [23:24:36] 6Mobile-Web: Bug with border parameter for tables - https://phabricator.wikimedia.org/T92913#1123780 (10Krenair) [23:25:01] dr0ptp4kt: I'm happy to check in w team at standup tomorrow AM on those tasks ^, your sugestions sound right to me. [23:25:03] kristenlans: https://phabricator.wikimedia.org/T91788 i think is appropriate for sprint 53, although do you want to email on mobile-tech to see if that should be a feature of sprint 53? it makes sense that it is part of sprint 53 so that junk doesn't get uploaded for internal testers [23:25:14] kristenlans: good [23:25:17] kristenlans: thx [23:25:26] 6Mobile-Web: Bug with border parameter for tables - https://phabricator.wikimedia.org/T92913#1123782 (10Tvx1) [23:26:06] kristenlans: i changed my mind about https://phabricator.wikimedia.org/T90981. the server is up and running and realistically the other tasks are captured as discrete tasks. so i think it's safe to mark it as resolved in sprint 52 [23:26:12] (03PS1) 10Robmoen: WIP: Article reader sees guidance about creating a collection [extensions/Gather] - 10https://gerrit.wikimedia.org/r/197247 [23:26:15] kristenlans: ok? want me to do that? [23:26:24] dr0ptp4kt: yup, soudns good :-) [23:26:44] 10Wikipedia-App-iOS-App, 3Mobile App Sprint 52 - iOS: EPIC: Setup Xcode Server - https://phabricator.wikimedia.org/T90981#1123788 (10dr0ptp4kt) [23:27:09] 10Wikipedia-App-iOS-App, 3Mobile App Sprint 52 - iOS: EPIC: Setup Xcode Server - https://phabricator.wikimedia.org/T90981#1123790 (10dr0ptp4kt) a:5dr0ptp4kt>3Fjalapeno [23:27:40] 10Wikipedia-App-iOS-App, 3Mobile App Sprint 52 - iOS: EPIC: Setup Xcode Server - https://phabricator.wikimedia.org/T90981#1072231 (10dr0ptp4kt) @Fjalapeno did this already. Other tasks have separate cards. Marking as resolved. [23:27:48] 10Wikipedia-App-iOS-App, 3Mobile App Sprint 52 - iOS: EPIC: Setup Xcode Server - https://phabricator.wikimedia.org/T90981#1123792 (10dr0ptp4kt) 5Open>3Resolved [23:27:56] Yeah I think until next month [23:28:45] dr0ptp4kt: ^ [23:29:32] coreyfloyd: word [23:36:49] 10Wikipedia-App-iOS-App: Build and test all Gerrit patch sets - https://phabricator.wikimedia.org/T91790#1123801 (10KLans_WMF) [23:37:40] 10Wikipedia-App-iOS-App: [Blocked] Build and distribute beta build to external testers weekly - https://phabricator.wikimedia.org/T91791#1123803 (10KLans_WMF) [23:44:30] 10Gather: Placeholder: I see a share button at the top of the list - https://phabricator.wikimedia.org/T91002#1123827 (10JKatzWMF) [23:47:52] 10Wikipedia-App-Android-App: Images' metadata labels do not have spaces between words in 'More information' - https://phabricator.wikimedia.org/T86810#1123838 (10Etonkovidova) [23:48:22] 10Wikipedia-App-Android-App: Images' metadata labels do not have spaces between words in 'More information' - https://phabricator.wikimedia.org/T86810#1123841 (10Etonkovidova) a:5Etonkovidova>3None [23:52:41] (03CR) 10Dr0ptp4kt: [C: 031 V: 031] "This is working for me." [apps/ios/wikipedia] - 10https://gerrit.wikimedia.org/r/197006 (owner: 10Mhurd)