[00:20:48] !log re-enabled both recurring charge jobs [00:20:55] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [07:35:27] (CR) jerkins-bot: [V: -1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/529008 (owner: L10n-bot) [07:48:20] (CR) jenkins-bot: Localisation updates from https://translatewiki.net. [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/529002 (owner: L10n-bot) [11:44:52] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM: Civi: harmonize currency & amount in TY emails - https://phabricator.wikimedia.org/T220636 (Ejegg) The back end has a function that takes currency code, amount, and locale, and formats them according to the customary display of that currency in that loc... [12:00:05] (CR) Ejegg: "looks like there's an unbalanced HTML tag" (1 comment) [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/529008 (owner: L10n-bot) [13:33:09] fundraising-tech-ops: puppetize dump_database cron job on fundraising build servers - https://phabricator.wikimedia.org/T228509 (Jgreen) Open→Resolved done [14:14:58] (CR) Ejegg: [C: +2] Update Tax, FAQ, and Donor privacy policy links [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/528921 (https://phabricator.wikimedia.org/T229457) (owner: Pcoombe) [14:40:17] (PS1) Ejegg: Parameterize max USD amount in bigamount message [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/529106 (https://phabricator.wikimedia.org/T229932) [14:48:30] Fundraising Sprint Princess Mongodb, Fundraising-Backlog, FR-Ingenico: Follow up re: Performance of Ingenico MyCheckout hosted pages - https://phabricator.wikimedia.org/T228953 (Ejegg) Replied to that email on July 25th with details of our hosted page customization and a sample setup API call. Matthe... [15:12:47] (Merged) jenkins-bot: Update Tax, FAQ, and Donor privacy policy links [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/528921 (https://phabricator.wikimedia.org/T229457) (owner: Pcoombe) [15:31:59] (PS2) Ejegg: Parameterize max USD amount in bigamount message [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/529106 (https://phabricator.wikimedia.org/T229932) [15:43:06] Fundraising Sprint Princess Mongodb, Fundraising-Backlog, MediaWiki-extensions-CentralNotice, Patch-For-Review, Security: CentralNotice setting a surprising content security policy in production when using &banner= URL parameter - https://phabricator.wikimedia.org/T225261 (Ejegg) p:Triage... [15:46:20] Wikimedia-Fundraising-Banners, donate.wikimedia.org: Can't use comma as decimal separator for Other amount in some banners - https://phabricator.wikimedia.org/T217854 (Pcoombe) Open→Resolved Changed all current best banners to use type="text" and inputmode="decimal", since this is supported in Ch... [15:49:35] Fundraising Sprint Princess Mongodb, Fundraising-Backlog: rando spanish page in India flow - https://phabricator.wikimedia.org/T229006 (XenoRyet) a:XenoRyet [15:49:47] Fundraising Sprint Princess Mongodb, Fundraising-Backlog: Dlocal cancel buttons don't go back to the payments wiki form - https://phabricator.wikimedia.org/T229336 (XenoRyet) a:XenoRyet [16:32:38] Wikimedia-Fundraising-Banners, donate.wikimedia.org: Can't use comma as decimal separator for Other amount in some banners - https://phabricator.wikimedia.org/T217854 (Ejegg) Looks like this might be nice to do on payments-wiki too, for cases when the amount box is editable [16:51:55] (PS1) Ejegg: WIP: allow , as decimal separator in amount [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/529120 (https://phabricator.wikimedia.org/T217854) [17:45:47] Wikimedia-Fundraising: Planned Giving assets for Thank You page - https://phabricator.wikimedia.org/T230148 (spatton) [18:34:00] fr-tech, looks like I no longer have the kafka role enabled afterall but kafkaserver is still filling up my logs [18:34:15] guess I'll have to work out what got installed and close it down the old fasioned way! [18:34:43] I wonder if disabling the role isn't enough to shut it down for some reason. [18:35:22] looks that way [19:10:44] Fundraising Sprint Princess Mongodb, Fundraising-Backlog: Dlocal cancel buttons don't go back to the payments wiki form - https://phabricator.wikimedia.org/T229336 (XenoRyet) Yea, this is on dLocal's side. Looks like for credit cards, back when fixing T131401, they changed something on their end to ki... [19:43:24] AndyRussG: are you able to run QUnit tests in debug mode? I'm getting a ton of 404 errors when I try adding debug=true on Special:JavaScriptTest [19:44:30] ah, looks like everything under the core /tests path - it's probably my server setup [19:46:16] ejegg: hmmm.... dunno, lemme check [19:46:58] One of the changes in the patch there was because of a CI-like check from a core test [19:47:14] (the removal of the configurability setting for the data property) [19:49:11] ejegg: ah hmm yeah still getting a weird error with a core test [19:49:15] or three [19:49:31] but if I just activate the CN QUnit test modules, it passes fine [19:49:55] ejegg: seems to pass everything in the CI setup, which is where it counts [19:50:54] also fr-tech any thoughts about the SWAT deploy? The last one of the week is at 4 pm Pacific [19:51:02] If folks want, I can just take care of it [19:51:48] AndyRussG: I signed up for it for the config updates on the Deployments page [19:52:10] RoanKattouw says he's not going to be able to, but maybe MaxSem or Niharika will be around [19:52:34] AndyRussG: OK, after adding a line to my nginx site config, it serves up the tests dir just fine [19:52:49] and running the tests in debug mode, all the ext.centralNotice ones pass [19:53:08] (and all appear!) [19:53:42] oh hey, even in non-debug mode they all pass now [19:53:55] ejegg: wrt SWAT oh nice! sorry, hadn't noticed... [19:54:00] I guess that missing nginx line was a problem for non-debug test runs as well [19:56:19] (PS2) Ejegg: Fix and reorganize QUnit tests [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/527861 (https://phabricator.wikimedia.org/T227879) (owner: AndyRussG) [19:56:54] (CR) Ejegg: [C: +2] "Looks good locally in debug and nondebug mode, and in CI. Thanks, AndyRussG!" [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/527861 (https://phabricator.wikimedia.org/T227879) (owner: AndyRussG) [20:05:46] (Merged) jenkins-bot: Fix and reorganize QUnit tests [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/527861 (https://phabricator.wikimedia.org/T227879) (owner: AndyRussG) [20:12:46] (CR) jenkins-bot: Fix and reorganize QUnit tests [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/527861 (https://phabricator.wikimedia.org/T227879) (owner: AndyRussG) [20:13:20] Fundraising Sprint Princess Mongodb, Fundraising-Backlog: rando spanish page in India flow - https://phabricator.wikimedia.org/T229006 (XenoRyet) This is on dLocal's side. The page at https://pay.dlocal.com/apd/retorno contains the text 'wait while we redirect you' and redirects to us, where we go to ei... [21:03:16] ejegg: thanks! ^ [21:06:12] yw! [21:06:45] btw trying to convince MaxSem the pages04 patch is kosher over in -releng [21:07:55] heheheh [21:08:00] maybe it isn't! [21:08:14] but it isn't removable for now [21:36:00] Fundraising Sprint Princess Mongodb, Fundraising-Backlog, FR-Email: Figure out if pymysql changes output format of silverpop export - https://phabricator.wikimedia.org/T230168 (Ejegg) [21:43:31] Fundraising-Backlog: Add PayTM to Dlocal India form - https://phabricator.wikimedia.org/T230170 (EMartin) [21:51:58] (CR) Ejegg: [C: +2] Port LocationType api from upstream so we can use for target smart [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/528616 (https://phabricator.wikimedia.org/T228712) (owner: Eileen) [21:54:10] (CR) Ejegg: [C: -1] "Looks like the tests already had the dotted format here, and this patch adds a second dot. Let's leave the |recur bits for now since they " [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/527179 (owner: Jgleeson) [21:56:27] (PS5) Ejegg: Deal with gateway_txn_id in a standard way [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/524535 [21:57:11] (Merged) jenkins-bot: Port LocationType api from upstream so we can use for target smart [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/528616 (https://phabricator.wikimedia.org/T228712) (owner: Eileen) [22:04:03] Fundraising Sprint Princess Mongodb, Fundraising-Backlog, FR-Email: Figure out if pymysql changes output format of silverpop export - https://phabricator.wikimedia.org/T230168 (Ejegg) Currently running an export on frdev against the dev_civicrm and dev_drupal databases, with the pymysql library. Sinc... [22:34:18] (CR) Ejegg: "To make this work I have to update the platform to 7.0.33 in the root composer.json. We should do that anyway, and I can't update it witho" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/526573 (owner: Eileen) [22:35:15] (CR) Ejegg: "recheck" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/526580 (https://phabricator.wikimedia.org/T228711) (owner: Eileen) [22:39:59] (CR) jerkins-bot: [V: -1] Update composer - league/csv version 9 [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/526580 (https://phabricator.wikimedia.org/T228711) (owner: Eileen) [23:49:10] Fundraising Sprint Office  , Fundraising Sprint Princess Mongodb, Fundraising-Backlog, Fr-CentralNotice-Caching: PHPUnit tests failing with "Cannot write with role DB_REPLICA" - https://phabricator.wikimedia.org/T229194 (AndyRussG) Open→Resolved Done!