[00:53:09] (PS1) Ejegg: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - https://gerrit.wikimedia.org/r/291147 [00:53:27] (CR) Ejegg: [V: 2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - https://gerrit.wikimedia.org/r/291147 (owner: Ejegg) [00:53:55] (CR) Ejegg: [C: 2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - https://gerrit.wikimedia.org/r/291147 (owner: Ejegg) [01:01:31] (PS1) Ejegg: Update DonationInterface submodule [core] (fundraising/REL1_25) - https://gerrit.wikimedia.org/r/291148 [01:01:48] (CR) Ejegg: [C: 2] Update DonationInterface submodule [core] (fundraising/REL1_25) - https://gerrit.wikimedia.org/r/291148 (owner: Ejegg) [01:07:59] (PS1) Ejegg: Fix CSS selector in Adyen iframe [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/291149 (https://phabricator.wikimedia.org/T135416) [01:08:17] (Merged) jenkins-bot: Update DonationInterface submodule [core] (fundraising/REL1_25) - https://gerrit.wikimedia.org/r/291148 (owner: Ejegg) [01:18:54] !log updated payments from e6807395d7687d521070b83d159b77b242e5c04f to de0398a244094f2bad6bc70eefce8388a616e575 [01:19:00] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log, Master [01:25:06] Fundraising-Backlog: Update vagrant main page test links - https://phabricator.wikimedia.org/T136379#2332721 (Ejegg) [01:35:40] Fundraising Sprint Jabberwock Slaying, Fundraising Sprint Killing Time, Fundraising-Backlog, Patch-For-Review: CentralNotice: test registration - https://phabricator.wikimedia.org/T134286#2260911 (Ejegg) @Pcoombe this is ready to play with on production! Also, reading web informs me that there's... [01:43:23] (PS1) Ejegg: Make Capture IPN logging like Auth IPN logging [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/291150 [02:14:28] (PS1) Ejegg: Quit flooding antifraud queue [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/291151 (https://phabricator.wikimedia.org/T136381) [02:25:36] (PS2) Ejegg: Quit flooding antifraud queue [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/291151 (https://phabricator.wikimedia.org/T136381) [02:31:13] phooey, nobody to CR... guess I'll roll back to stop the failmail [02:33:20] !log rolled back payments from de0398a244094f2bad6bc70eefce8388a616e575 to e6807395d7687d521070b83d159b77b242e5c04f [02:33:26] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log, Master [02:46:34] (PS3) Ejegg: Rename hook functions [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290829 (https://phabricator.wikimedia.org/T136381) [02:46:47] (PS3) Ejegg: Comment and DRY for functions filter [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290830 (https://phabricator.wikimedia.org/T136381) [02:46:59] (PS4) Ejegg: Const and better name for initial ip velocity flag [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290990 (https://phabricator.wikimedia.org/T136381) [03:19:27] (PS1) Ejegg: Trim fields before sending to server [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/291154 [03:31:40] Fundraising Sprint Ghostbusting , Fundraising Sprint Hermit Crab Husbandry, Fundraising Sprint Internet Exploring, Fundraising Sprint Jabberwock Slaying, and 3 others: Only subscribe primary emails, secondary addresses should be suppressed - https://phabricator.wikimedia.org/T131979#2332862 (Ejegg... [03:48:07] (PS1) Ejegg: NumberFormatter fallback when ext-intl not installed [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/291156 [03:49:34] man, i shouldn't drink coffee after noon... [03:52:45] (PS6) Ejegg: WIP fraud points for email address at nonexistant tld [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290615 [03:53:03] (CR) jenkins-bot: [V: -1] WIP fraud points for email address at nonexistant tld [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290615 (owner: Ejegg) [04:58:00] (CR) Ejegg: "Order is tricky here - command-line options include specifying an alternate config file and section, so you need to parse those before you" [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/288314 (owner: Ejegg) [07:44:27] Fundraising Sprint Hermit Crab Husbandry, Fundraising Sprint Internet Exploring, Fundraising Sprint Jabberwock Slaying, Fundraising Sprint Killing Time, and 4 others: [Epic] Adyen form for French (France), Japanese (Japan), Ukrainian (Ukraine), & He... - https://phabricator.wikimedia.org/T128812#2333006 [10:00:44] (CR) Bartosz Dziewoński: "Caused T136387." [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/290361 (https://phabricator.wikimedia.org/T134286) (owner: AndyRussG) [11:23:02] Fundraising Sprint Hermit Crab Husbandry, Fundraising Sprint Internet Exploring, Fundraising-Backlog, MediaWiki-extensions-CentralNotice, Patch-For-Review: mw.centralNotice CRASH on client side - https://phabricator.wikimedia.org/T133765#2333354 (PerfektesChaos) @AndyRussG – see T136387 – Fun... [14:01:34] Fundraising-Backlog, FR-Adyen: Adyen in Ukrainian in Ukraine - https://phabricator.wikimedia.org/T124228#2333878 (Ata) @DStrine, everything is translated on the links you mentioned so I guess the problem is not there. I was talking about the file attached to the description of this task. Someone (even me... [15:04:28] Fundraising Sprint Jabberwock Slaying, Fundraising Sprint Killing Time, Fundraising-Backlog, Patch-For-Review: CentralNotice: test registration - https://phabricator.wikimedia.org/T134286#2260911 (matmarex) This caused T136387 and the production deployment was just reverted. [15:05:11] Fundraising-Backlog, MediaWiki-extensions-CentralNotice, Browser-Support-Internet-Explorer, JavaScript: EcmaScript 6 features are not supported in older browsers - https://phabricator.wikimedia.org/T136387#2332918 (matmarex) a:matmarex>None Deployment of the faulty patch (T134286) was rev... [15:29:33] Fundraising-Backlog, MediaWiki-extensions-CentralNotice, Browser-Support-Internet-Explorer, JavaScript: EcmaScript 6 features are not supported in older browsers - https://phabricator.wikimedia.org/T136387#2334153 (Strainu) I can confirm JavaScript now works in IE 11. Waiting for confirmation on... [15:49:20] Fundraising-Backlog, MediaWiki-extensions-CentralNotice, Browser-Support-Internet-Explorer, JavaScript: EcmaScript 6 features are not supported in older browsers - https://phabricator.wikimedia.org/T136387#2334209 (Antilope) I can confirm JavaScript now works again in Opera 12.18 (released Feb 20... [15:53:46] Fundraising Sprint Jabberwock Slaying, Fundraising Sprint Killing Time, Fundraising-Backlog, Patch-For-Review: CentralNotice: test registration - https://phabricator.wikimedia.org/T134286#2334228 (DStrine) p:Triage>High [16:06:26] Fundraising-Backlog, MediaWiki-extensions-CentralNotice, Browser-Support-Internet-Explorer, JavaScript: EcmaScript 6 features are not supported in older browsers - https://phabricator.wikimedia.org/T136387#2332918 (Krinkle) >>! In T136387#2334101, @matmarex wrote: > Deployment of the faulty patch... [16:06:47] Fundraising-Backlog, MediaWiki-extensions-CentralNotice, Browser-Support-Internet-Explorer, JavaScript: CentralNotice failing in older browsers due use of ECMAScript 6 syntax - https://phabricator.wikimedia.org/T136387#2334287 (Krinkle) [16:07:41] Fundraising-Backlog, MediaWiki-extensions-CentralNotice, Browser-Support-Internet-Explorer, JavaScript: CentralNotice failing in older browsers due use of ECMAScript 6 syntax - https://phabricator.wikimedia.org/T136387#2334289 (matmarex) >>! In T136387#2334284, @Krinkle wrote: > It should be coin... [16:07:43] Fundraising-Backlog, MediaWiki-extensions-CentralNotice, JavaScript: Introduce automatic ECMA version check on JS patch review - https://phabricator.wikimedia.org/T136408#2333737 (matmarex) [16:08:58] Fundraising-Backlog, MediaWiki-extensions-CentralNotice, JavaScript: Update CentralNotice JSHint config to restrict syntax to ES3 (disallow ES5 or ES6) - https://phabricator.wikimedia.org/T136408#2334303 (Krinkle) [16:40:15] Fundraising Sprint Ghostbusting , Fundraising Sprint Hermit Crab Husbandry, Fundraising Sprint Internet Exploring, Fundraising Sprint Jabberwock Slaying, and 3 others: Only subscribe primary emails, secondary addresses should be suppressed - https://phabricator.wikimedia.org/T131979#2334409 (CCogd... [16:41:44] Fundraising Sprint Ghostbusting , Fundraising Sprint Hermit Crab Husbandry, Fundraising Sprint Internet Exploring, Fundraising Sprint Jabberwock Slaying, and 3 others: Only subscribe primary emails, secondary addresses should be suppressed - https://phabricator.wikimedia.org/T131979#2334412 (MBeat... [16:43:21] (PS1) Ejegg: WIP jshint and gruntfile fixes [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291264 (https://phabricator.wikimedia.org/T136408) [16:45:41] (CR) jenkins-bot: [V: -1] WIP jshint and gruntfile fixes [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291264 (https://phabricator.wikimedia.org/T136408) (owner: Ejegg) [16:46:33] oh hey ejegg are you trying to help with the CN rollback thing? I was just reading up on those tasks [16:46:38] yeah [16:46:50] it breaks old browsers [16:47:10] and we haven't actually been checking most of the js files in continuous integration [16:47:23] thanks! you're like the green beret these last 2 weeks [16:47:28] so ^^ was turning on the tests, including enforcing old syntax [16:47:33] and... lots of breakage [16:47:59] hopefully mostly trivial fixes and nothing that will change logic [16:48:07] heh, thanks [16:48:18] This could wait for Andy and Adam to get back ... it's up to you [16:48:30] ehh, I'll pounce on it [16:48:43] unless we need to respond to part of it today [16:48:45] ok [16:48:46] haven't peeked at CN for a while, I'll take this opportunity [16:49:22] ok [17:12:12] (PS2) Ejegg: jshint, gruntfile, and compatibility fixes [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291264 [17:27:37] (PS3) Ejegg: jshint, gruntfile, and compatibility fixes [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291264 [17:31:40] (PS4) Ejegg: jshint, gruntfile, and compatibility fixes [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291264 [17:40:36] Fundraising-Backlog, MediaWiki-extensions-CentralNotice, Tracking: CentralNotice: reducing initial JS load? (tracking) - https://phabricator.wikimedia.org/T133741#2334627 (Danny_B) [18:01:25] (PS1) Ejegg: WIP enforce jscs, make it pass [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291288 [18:50:35] (PS2) Ejegg: Enforce jscs, make it pass [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291288 [18:55:26] (CR) Bartosz Dziewoński: [C: 2] jshint, gruntfile, and compatibility fixes [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291264 (owner: Ejegg) [18:56:57] (CR) Bartosz Dziewoński: "Hmm. I think you need to move the .jscsrc to the next patch." [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291264 (owner: Ejegg) [18:57:16] (CR) Ejegg: "oops, will do" [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291264 (owner: Ejegg) [18:58:00] (CR) jenkins-bot: [V: -1] jshint, gruntfile, and compatibility fixes [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291264 (owner: Ejegg) [18:58:21] (CR) Bartosz Dziewoński: "(And the related Gruntfile and package.json changes)" [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291264 (owner: Ejegg) [18:59:27] (CR) Ejegg: "yeah, must have --amended by mistake at some point" [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291264 (owner: Ejegg) [19:01:45] (PS5) Ejegg: jshint, gruntfile, and compatibility fixes [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291264 [19:05:57] Fundraising-Backlog, MediaWiki-extensions-CentralNotice, Browser-Support-Internet-Explorer, JavaScript, and 2 others: CentralNotice failing in older browsers due use of ECMAScript 6 syntax - https://phabricator.wikimedia.org/T136387#2334954 (Trizek-WMF) [19:15:15] dstrine: You WFH? [19:15:33] (PS3) Ejegg: Enforce jscs, make it pass [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291288 [19:22:28] (PS4) Ejegg: Enforce jscs, make it pass [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291288 [19:24:06] (CR) Ejegg: "thanks Krinkle, guess I leaned a little too heavily on autotune. I'll review the rest and revert any other overzealousness" [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291288 (owner: Ejegg) [19:26:25] (CR) Bartosz Dziewoński: [C: 2] jshint, gruntfile, and compatibility fixes [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291264 (owner: Ejegg) [19:28:13] (Merged) jenkins-bot: jshint, gruntfile, and compatibility fixes [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291264 (owner: Ejegg) [19:31:26] awight: yeah [19:31:33] Fundraising-Backlog, MediaWiki-extensions-CentralNotice, JavaScript: Update CentralNotice JSHint config to restrict syntax to ES3 (disallow ES5 or ES6) - https://phabricator.wikimedia.org/T136408#2335053 (matmarex) Open>Resolved a:Ejegg [19:31:56] Fundraising-Backlog, MediaWiki-extensions-CentralNotice, Browser-Support-Internet-Explorer, JavaScript, User-notice: CentralNotice failing in older browsers due use of ECMAScript 6 syntax - https://phabricator.wikimedia.org/T136387#2335056 (matmarex) Open>Resolved a:Ejegg [19:32:37] dstrine: Do you know if eileen is in the office today? [19:32:59] Fundraising Sprint Killing Time, Fundraising-Backlog, MediaWiki-extensions-CentralNotice, Browser-Support-Internet-Explorer, and 3 others: CentralNotice failing in older browsers due use of ECMAScript 6 syntax - https://phabricator.wikimedia.org/T136387#2335060 (Ejegg) [19:33:03] awight: it doesn't look like it. She isn't usually [19:33:08] uhh [19:33:10] wat [19:33:15] on fridays? [19:33:20] it's her saturday [19:33:25] But she's here, I thought [19:33:29] do I have the week wrong? [19:33:42] Fundraising Sprint Killing Time, Fundraising-Backlog, MediaWiki-extensions-CentralNotice, JavaScript, Unplanned-Sprint-Work: Update CentralNotice JSHint config to restrict syntax to ES3 (disallow ES5 or ES6) - https://phabricator.wikimedia.org/T136408#2335062 (Ejegg) [19:34:04] * awight facepalms all over the place [19:34:14] I don't wanna talk about it. [19:34:15] awight: the schedule says 6/9 and 6/10 [19:34:20] Well, at least I didn't miss her yesterday [19:34:24] LOL [19:38:38] Wikimedia-Fundraising: Create banners for HoverCards A/B test - https://phabricator.wikimedia.org/T134291#2261096 (awight) Please note the comments on T134286 -- there are some special "legacy" campaign settings you'll need to enable in order to use my workaround, and sample code that comes pretty close to w... [19:39:05] * awight becomes increasingly agitated [19:39:58] Anybody have things I can help with? [19:40:18] (CR) Awight: [C: 2] "Debt writeoff!" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/289716 (https://phabricator.wikimedia.org/T122411) (owner: Ejegg) [19:40:19] awight: there's something to limit the antifraud spam [19:40:21] (PS2) Awight: Remove old email edit hook [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/289716 (https://phabricator.wikimedia.org/T122411) (owner: Ejegg) [19:40:51] (CR) Awight: [C: 2] Keep Civi users off suppression list [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/291136 (https://phabricator.wikimedia.org/T122411) (owner: Ejegg) [19:40:53] (CR) jenkins-bot: [V: -1] Remove old email edit hook [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/289716 (https://phabricator.wikimedia.org/T122411) (owner: Ejegg) [19:41:17] hrm? [19:41:34] cloning bug [19:41:34] ah, db issue [19:41:51] (CR) Ejegg: "recheck" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/289716 (https://phabricator.wikimedia.org/T122411) (owner: Ejegg) [19:42:00] ah the cloning issue was a warning [19:43:13] (Merged) jenkins-bot: Keep Civi users off suppression list [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/291136 (https://phabricator.wikimedia.org/T122411) (owner: Ejegg) [19:44:39] hah! Phabricator double-encodes and breaks its own link [19:44:48] Must be part of the "workflow" [19:51:00] (CR) Awight: [C: 2] "It's an improvement, but these classes have quite an existing funky smell. Time for a bigger cleanup!" (3 comments) [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/291151 (https://phabricator.wikimedia.org/T136381) (owner: Ejegg) [19:51:46] (PS4) Awight: Rename hook functions [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290829 (https://phabricator.wikimedia.org/T136381) (owner: Ejegg) [19:51:54] (CR) Awight: [C: 2] Rename hook functions [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290829 (https://phabricator.wikimedia.org/T136381) (owner: Ejegg) [19:52:27] (CR) Awight: [C: 2] Comment and DRY for functions filter [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290830 (https://phabricator.wikimedia.org/T136381) (owner: Ejegg) [19:52:33] (PS4) Awight: Comment and DRY for functions filter [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290830 (https://phabricator.wikimedia.org/T136381) (owner: Ejegg) [19:53:18] (Merged) jenkins-bot: Quit flooding antifraud queue [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/291151 (https://phabricator.wikimedia.org/T136381) (owner: Ejegg) [19:53:36] (Merged) jenkins-bot: Rename hook functions [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290829 (https://phabricator.wikimedia.org/T136381) (owner: Ejegg) [19:55:08] (CR) Awight: [C: 2] "Beneficial protectionism!" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290982 (owner: Ejegg) [19:55:12] (PS3) Awight: Rationalize accesibility of fns and methods for Extras [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290982 (owner: Ejegg) [19:55:48] (CR) jenkins-bot: [V: -1] Rationalize accesibility of fns and methods for Extras [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290982 (owner: Ejegg) [19:56:13] (CR) Awight: [C: 2] "Thanks!" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290990 (https://phabricator.wikimedia.org/T136381) (owner: Ejegg) [19:56:27] (PS5) Awight: Comment and DRY for functions filter [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290830 (https://phabricator.wikimedia.org/T136381) (owner: Ejegg) [19:56:46] (CR) Awight: "@ejegg aww, manual rebase needed." [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290982 (owner: Ejegg) [19:58:10] (CR) Ejegg: "dang, really hoped to avoid that for this guy!" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290982 (owner: Ejegg) [20:02:29] (PS9) Awight: PayPal Express Checkout: recurring [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/287036 (https://phabricator.wikimedia.org/T134446) [20:02:43] (CR) Awight: "PS9: Manual rebase" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/287036 (https://phabricator.wikimedia.org/T134446) (owner: Awight) [20:02:57] (PS3) Awight: Stop sending item category [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/288253 (https://phabricator.wikimedia.org/T134446) [20:03:16] ejegg: recurring is ready for CR if you get the chance... [20:04:18] cool! [20:04:42] (CR) Awight: [C: 2] "Confirmed unused." [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/288314 (owner: Ejegg) [20:04:54] (PS2) Awight: replace busted addOptionWithConfigDefault [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/288314 (owner: Ejegg) [20:04:58] ejegg: and awight ... ejegg and I are the only official people left here today. I don't think we NEED a standup unless you want to chat about anything [20:05:00] (PS4) Awight: Fix defaults for QueueJobRunner [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/287871 (owner: Ejegg) [20:06:06] ejegg: we don't have to meet [20:06:16] It's technically just us [20:06:31] I'm happy to chat, but probably leaving soon. [20:06:36] My official objectives have been foiled [20:06:56] (CR) Awight: [C: 2] Fix defaults for QueueJobRunner (1 comment) [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/287871 (owner: Ejegg) [20:07:01] (PS6) Awight: Fix time and message limits [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/288449 (https://phabricator.wikimedia.org/T133965) (owner: Ejegg) [20:07:13] ok, a/v issues here anyway [20:07:27] my report: messed with a bunch of little fixes last night [20:07:39] ok [20:07:47] including something for the reason i had to roll back [20:07:54] then today mostly central notice [20:08:21] ah ... are you in on monday? I canceled the standup becuase I will not be and I assumed no one else was [20:08:33] dstrine: not planning to be [20:08:38] ok great :) [20:08:50] Wow, thanks for the cleanup btw, I was really hoping to not leave you with a third-hand disaster :( [20:08:51] oh... i might have forgotten it was a holiday in some email [20:08:57] I canceled that meeting without asking anyone [20:09:12] (CR) Awight: [C: 2] Fix time and message limits (1 comment) [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/288449 (https://phabricator.wikimedia.org/T133965) (owner: Ejegg) [20:09:27] I have no need to meet on Monday either, if that's the question :p [20:09:29] awight: third-hand? I think that was all my code ringing the alarm bells [20:09:31] (PS7) Awight: Consume pending queue to database [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/287147 (https://phabricator.wikimedia.org/T133197) (owner: Ejegg) [20:09:42] (Merged) jenkins-bot: replace busted addOptionWithConfigDefault [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/288314 (owner: Ejegg) [20:09:44] (Merged) jenkins-bot: Fix defaults for QueueJobRunner [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/287871 (owner: Ejegg) [20:10:02] whew, looks like I'll have a few code comments to folow up on [20:10:03] ejegg: I thought that was AndyRussG's patch, which I butchered and self-merged [20:10:12] thanks for the blitz! [20:10:12] the registerTest fail [20:10:19] awight: ohh, thataone [20:10:25] yah sorry about the binge and purge CR [20:10:28] i mean, the real fail was in the gruntfile [20:10:35] (CR) jenkins-bot: [V: -1] Fix time and message limits [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/288449 (https://phabricator.wikimedia.org/T133965) (owner: Ejegg) [20:10:52] it was doing jshint on... the gruntfile [20:10:58] and nothing else [20:11:21] omgwow [20:11:22] (CR) jenkins-bot: [V: -1] Consume pending queue to database [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/287147 (https://phabricator.wikimedia.org/T133197) (owner: Ejegg) [20:11:48] btw .......PHP Fatal error: Class 'SmashPig\Tests\Context' not found in /mnt/jenkins-workspace/workspace/composer-php53@2/Tests/QueueConsumerTest.php on line 146 [20:11:57] Missing namespace I guess [20:12:43] oops, lemme fix that [20:12:45] hey awight and ejegg I'm palying with sql in civi and I'm super new. I just want to try looking different things up. Is there a way to get a list of tables, their column headers and the types of data in the columns? I want to try looking for things but it's kinda hard when you don't know what to search for. This question might just be me being a total noob so applogies in [20:12:45] advance. [20:13:03] dstrine: show tables; [20:13:13] show create table FOO [20:13:26] or for a subset: show tables like '%contact%'; [20:13:37] show table status; [20:13:40] describe table FOO; [20:14:11] less verbose alternate to show create table foo; is "describe foo;" [20:14:15] dstrine: https://collab.wikimedia.org/wiki/Fundraising/Engineering/SQL_Database_Overview [20:14:18] what he said^ [20:14:50] We should really start using SQL comments... so column descriptions show up in the above commands. [20:14:59] s/in/using/ [20:15:14] yah [20:15:41] awight: That link is awesome. I've asked like 5 people about that and they just laughed me :( [20:15:58] ok ... like 3 people really [20:16:01] (PS5) Ejegg: Enforce jscs, make it pass [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291288 [20:17:05] hrm [20:17:15] We should move it to the public wiki actually. nothing to hide there. [20:17:29] And I can never find the little bastard due to search [20:17:53] awight: pgehres is not related to civi, right? [20:18:23] sigh [20:18:35] pgehres is the Django database for storing banner impressions from S:RI [20:21:18] Totally on my hit list at the end of this queue refactoring [20:27:30] (CR) Awight: [C: -1] "This is *really* great. I made a few suggestions inline..." (9 comments) [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/287147 (https://phabricator.wikimedia.org/T133197) (owner: Ejegg) [20:28:54] (PS6) Ejegg: Enforce jscs, make it pass [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291288 [20:32:38] gtg! [20:32:41] have a fun weekend [20:32:46] see ya, you too [20:37:05] (CR) Ejegg: "Krinkle: now with 75% less robotic mangling! I think it's ready for another pass if you want to take a look." [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/291288 (owner: Ejegg) [20:37:21] ok, lets see what awight had to say [20:37:34] *let's [20:42:41] (CR) Ejegg: "thanks for the comments. Re-reading, I realize this was panic-tinged code. Maybe we always send antifraud messages on the old filter hoo" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/291151 (https://phabricator.wikimedia.org/T136381) (owner: Ejegg) [20:43:31] (CR) Ejegg: Fix defaults for QueueJobRunner (1 comment) [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/287871 (owner: Ejegg) [20:46:09] (PS6) Ejegg: Comment and DRY for functions filter [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290830 (https://phabricator.wikimedia.org/T136381) [20:51:50] (CR) Ejegg: "d'oh - bad autorebase" [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/288449 (https://phabricator.wikimedia.org/T133965) (owner: Ejegg) [21:01:00] (PS7) Ejegg: Fix time and message limits [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/288449 (https://phabricator.wikimedia.org/T133965) [21:03:19] (CR) Ejegg: [C: 2] "Removing and re-applying awight's +2 to convince Zuul to merge" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290830 (https://phabricator.wikimedia.org/T136381) (owner: Ejegg) [21:04:44] (Merged) jenkins-bot: Comment and DRY for functions filter [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290830 (https://phabricator.wikimedia.org/T136381) (owner: Ejegg) [21:23:55] (PS4) Ejegg: Rationalize accesibility of fns and methods for Extras [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290982 [21:24:24] (CR) Ejegg: "PS4: manual rebase" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290982 (owner: Ejegg) [21:25:53] (PS5) Ejegg: Const and better name for initial ip velocity flag [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290990 (https://phabricator.wikimedia.org/T136381) [21:28:26] (PS5) Ejegg: Rationalize accesibility of fns and methods for Extras [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290982 [21:29:12] (CR) Ejegg: "PS5: pulled the TestingAccessWrapper up from following patch" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290982 (owner: Ejegg) [21:29:23] (PS6) Ejegg: Const and better name for initial ip velocity flag [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/290990 (https://phabricator.wikimedia.org/T136381) [21:36:16] (PS8) Ejegg: Consume pending queue to database [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/287147 (https://phabricator.wikimedia.org/T133197) [22:08:09] (PS9) Ejegg: Consume pending queue to database [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/287147 (https://phabricator.wikimedia.org/T133197) [22:11:06] (CR) Ejegg: Consume pending queue to database (9 comments) [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/287147 (https://phabricator.wikimedia.org/T133197) (owner: Ejegg) [22:42:55] (PS1) Ejegg: Simplify antifraud message logic [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/291367 (https://phabricator.wikimedia.org/T136381) [22:43:32] (PS2) Ejegg: Simplify antifraud message logic [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/291367 (https://phabricator.wikimedia.org/T136381) [22:44:24] (CR) Ejegg: "I6ce06002eee1f oughtta dissipate some of that funky odor" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/291151 (https://phabricator.wikimedia.org/T136381) (owner: Ejegg) [22:53:17] gonna skedaddle. so long! [23:02:51] (CR) Ejegg: "couple questions, i'll look at it more tuesday" (2 comments) [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/287036 (https://phabricator.wikimedia.org/T134446) (owner: Awight) [23:08:12] Fundraising Sprint Hermit Crab Husbandry, Fundraising Sprint Internet Exploring, Fundraising Sprint Jabberwock Slaying, Fundraising Sprint Killing Time, and 4 others: [Epic] Adyen form for French (France), Japanese (Japan), Ukrainian (Ukraine), & He... - https://phabricator.wikimedia.org/T128812#2335721 [23:16:56] I'm out too...