[00:39:45] (03PS5) 10Eileen: WIP: Move Damage table to CiviCRM extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/844072 (https://phabricator.wikimedia.org/T295412) (owner: 10Damilare Adedoyin) [00:45:21] (03PS1) 10Scardenasmolinar: Add PreferencesGetIconHook [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/849199 (https://phabricator.wikimedia.org/T317419) [00:47:29] (03CR) 10CI reject: [V: 04-1] Add PreferencesGetIconHook [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/849199 (https://phabricator.wikimedia.org/T317419) (owner: 10Scardenasmolinar) [00:52:47] (03CR) 10CI reject: [V: 04-1] WIP: Move Damage table to CiviCRM extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/844072 (https://phabricator.wikimedia.org/T295412) (owner: 10Damilare Adedoyin) [00:54:45] (03CR) 10Scardenasmolinar: "recheck" [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/849199 (https://phabricator.wikimedia.org/T317419) (owner: 10Scardenasmolinar) [01:01:02] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Decide a naming convention for our extensions & document on wikitech - https://phabricator.wikimedia.org/T321638 (10Eileenmcnaughton) [01:01:35] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Decide a naming convention for our extensions & document on wikitech - https://phabricator.wikimedia.org/T321638 (10Eileenmcnaughton) [01:05:22] (03CR) 10Eileen: "I found a file path issue causing some class loader issues (it was suprisingly hard to spot) & pushed it up." [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/838948 (https://phabricator.wikimedia.org/T308181) (owner: 10Ejegg) [01:14:54] (03PS6) 10Eileen: WIP: Move Damage table to CiviCRM extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/844072 (https://phabricator.wikimedia.org/T295412) (owner: 10Damilare Adedoyin) [01:16:11] (03CR) 10Eileen: "Just another minor fix - test fails were on" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/844072 (https://phabricator.wikimedia.org/T295412) (owner: 10Damilare Adedoyin) [01:31:43] (03PS7) 10Eileen: WIP: Move Damage table to CiviCRM extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/844072 (https://phabricator.wikimedia.org/T295412) (owner: 10Damilare Adedoyin) [01:32:23] (03CR) 10Eileen: "I got into a real muddle there with the patches - I think it's correct again now :-)" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/844072 (https://phabricator.wikimedia.org/T295412) (owner: 10Damilare Adedoyin) [01:41:07] (03PS1) 10Ejegg: Ignore non-successful lines in PayPal audit [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/849205 (https://phabricator.wikimedia.org/T320989) [01:42:49] (03CR) 10CI reject: [V: 04-1] Ignore non-successful lines in PayPal audit [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/849205 (https://phabricator.wikimedia.org/T320989) (owner: 10Ejegg) [01:46:37] (03CR) 10Ejegg: "To check the value of this column for the TRR files on production you can try something like" [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/849205 (https://phabricator.wikimedia.org/T320989) (owner: 10Ejegg) [01:47:29] (03PS2) 10Ejegg: Ignore non-successful lines in PayPal audit [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/849205 (https://phabricator.wikimedia.org/T320989) [02:13:09] (03PS1) 10Ejegg: Update Symfony/Yaml and fix maps and tests [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/849208 (https://phabricator.wikimedia.org/T321621) [02:13:50] 10Fundraising Sprint Turtles that are robotic that destroy the whole world with their foot theory, 10Fundraising-Backlog, 10FR-Docker, 10FR-Smashpig, 10Patch-For-Review: Update symfony\yaml library used in SmashPig - https://phabricator.wikimedia.org/T321621 (10Ejegg) [02:19:29] (03Abandoned) 10Ejegg: Namespace for ExchangeRatesException [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/701606 (owner: 10Ejegg) [05:07:08] (03CR) 10CI reject: [V: 04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_38) - 10https://gerrit.wikimedia.org/r/849262 (owner: 10L10n-bot) [05:24:15] (03PS1) 10Eileen: Port search kit fix [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849384 (https://phabricator.wikimedia.org/T295412) [05:26:39] (03PS8) 10Eileen: WIP: Move Damage table to CiviCRM extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/844072 (https://phabricator.wikimedia.org/T295412) (owner: 10Damilare Adedoyin) [05:43:59] 10Fundraising Sprint Turtles that are robotic that destroy the whole world with their foot theory, 10Fundraising Tech - Chaos Crew, 10Wikimedia-Fundraising-CiviCRM: Clean up our five character locale data.... - https://phabricator.wikimedia.org/T321251 (10AnnWF) [05:54:04] 10Fundraising Sprint Turtles that are robotic that destroy the whole world with their foot theory, 10Fundraising-Backlog, 10FR-PayPal-ExpressCheckout, 10FR-Smashpig: Implement PayPal EC CreateRecurringPaymentsProfile in SmashPig - https://phabricator.wikimedia.org/T318881 (10AnnWF) a:03AnnWF [07:00:59] (03CR) 10Eileen: [C: 03+2] "I gave this a spin & it works." [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/838980 (https://phabricator.wikimedia.org/T318529) (owner: 10Ejegg) [07:09:00] (03PS2) 10Eileen: Port search kit fix [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849384 (https://phabricator.wikimedia.org/T295412) [07:09:02] (03PS9) 10Eileen: WIP: Move Damage table to CiviCRM extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/844072 (https://phabricator.wikimedia.org/T295412) (owner: 10Damilare Adedoyin) [07:09:28] (03CR) 10Eileen: "Aha - just got `Damaged.get` to work again - by changing to extend `DAOEntity`" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/844072 (https://phabricator.wikimedia.org/T295412) (owner: 10Damilare Adedoyin) [07:32:23] (03CR) 10Thiemo Kreuz (WMDE): Make use of the ?? syntax where it makes sense (031 comment) [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/836802 (owner: 10Thiemo Kreuz (WMDE)) [10:51:39] (03CR) 10Nikerabbit: [V: 03+2] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_38) - 10https://gerrit.wikimedia.org/r/849262 (owner: 10L10n-bot) [13:52:19] 10Fundraising-Backlog, 10fr-donorservices: Civi: Forget Me button yields configuration error - https://phabricator.wikimedia.org/T321694 (10MBeat33) [13:59:07] 10Fundraising-Backlog, 10fr-donorservices: Civi: Forget Me button yields configuration error - https://phabricator.wikimedia.org/T321694 (10MBeat33) [14:09:21] (03CR) 10AndyRussG: "Heyy thanks for working on this!! Just to note, the inconsistent qunit test result you see above is not exclusive to this patch... so, rec" [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/849199 (https://phabricator.wikimedia.org/T317419) (owner: 10Scardenasmolinar) [14:21:52] (03CR) 10Jsn.sherman: [C: 03+1] "This looks mergable to me; thanks Susana!" [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/849199 (https://phabricator.wikimedia.org/T317419) (owner: 10Scardenasmolinar) [15:36:24] hi fr-tech! [15:37:14] cstone: any luck figuring out what the full responses looked like for those pending messages with no gateway_txn_id? [15:38:46] Hey ejegg I broke it into two tickets one for figuring out why it's happening and then one for the pending transaction resolver [15:38:55] I put the figuring out why on the chaos board [15:39:24] ok, cool. So for now you'll just filter them out in the isResolvable function? [15:39:51] I will dive into the figuring out [15:40:08] hmm, I guess after I add those unit tests I've been promising [15:40:11] Yeah since it's already checking for other things like that made sense to add another [15:40:17] Thanks ejegg [15:40:21] yep, sounds good to me [15:47:54] (03PS2) 10Ejegg: Work around name parser bug on multibyte initials [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/845102 (https://phabricator.wikimedia.org/T321343) [15:48:10] (03PS3) 10Ejegg: Work around name parser bug on multibyte initials [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/845102 (https://phabricator.wikimedia.org/T321343) [16:01:18] (03CR) 10Wfan: [C: 03+2] "Looks good thanks! 😊" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/845102 (https://phabricator.wikimedia.org/T321343) (owner: 10Ejegg) [16:04:42] (03CR) 10CI reject: [V: 04-1] Work around name parser bug on multibyte initials [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/845102 (https://phabricator.wikimedia.org/T321343) (owner: 10Ejegg) [16:14:43] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10donate.wikimedia.org: Broken links from Waystogive? - https://phabricator.wikimedia.org/T315268 (10AnnWF) a:05AnnWF→03None [16:14:55] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice, 10MediaWiki-extensions-Translate: Can’t mark CentralNotice translations as reviewed - https://phabricator.wikimedia.org/T308614 (10AnnWF) a:05AnnWF→03None [16:39:12] (03PS2) 10Ejegg: Avoid stack overflow in Contact ShowMe [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/844080 [16:40:08] (03PS4) 10Ejegg: Work around name parser bug on multibyte initials [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/845102 (https://phabricator.wikimedia.org/T321343) [16:53:44] (03CR) 10CI reject: [V: 04-1] Work around name parser bug on multibyte initials [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/845102 (https://phabricator.wikimedia.org/T321343) (owner: 10Ejegg) [16:55:09] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Pending table rows have no gateway_txn_id in the column and in the message its gateway_txn_id:false - https://phabricator.wikimedia.org/T321631 (10Ejegg) All of these (well, the most recent 10 that I checked) seem to either be rtbt or 3DS redirects, where... [16:56:00] (03PS5) 10Ejegg: Work around name parser bug on multibyte initials [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/845102 (https://phabricator.wikimedia.org/T321343) [16:59:47] (03PS8) 10Ejegg: Frontend email validation should block ones starting with dash [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/843501 (https://phabricator.wikimedia.org/T320570) (owner: 10Damilare Adedoyin) [16:59:54] (03CR) 10Ejegg: [C: 03+2] "Looks good!" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/843501 (https://phabricator.wikimedia.org/T320570) (owner: 10Damilare Adedoyin) [17:00:47] cstone does the explanation in the comment on https://phabricator.wikimedia.org/T321631 sound OK to you? [17:02:14] sorry, no need for synchronous answer, it's in review for when you want to look [17:02:14] yeah ejegg good to know [17:08:40] (03Merged) 10jenkins-bot: Frontend email validation should block ones starting with dash [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/843501 (https://phabricator.wikimedia.org/T320570) (owner: 10Damilare Adedoyin) [17:17:11] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Back again but from donations queue consumer: INVALID_MESSAGE Recurring donation, but no subscription ID or recurring payment token found - https://phabricator.wikimedia.org/T320587 (10Ejegg) This seems to have abated with the latest patch for order_id re... [17:24:30] 10Fundraising Tech - Chaos Crew: Fraud Attack - https://phabricator.wikimedia.org/T321608 (10EMartin) @TSkaff Hi Thea, can you help us escalate attention on this? I have info to share and this fraud pattern has now moved to AMEX. We need to block and I will post the info here when we move this task to private... [17:32:51] tryna figure out a 'detached HEAD' git halloween costume [18:16:21] dwisehaupt: are you free to join tech talk for some db troubleshooting? [18:16:27] https://meet.google.com/odz-juym-nog?authuser=0 [18:16:37] yeah. just a sec. [18:17:02] thanks! [18:40:31] (03CR) 10Fomafix: [C: 03+2] Make use of the ?? syntax where it makes sense (031 comment) [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/836802 (owner: 10Thiemo Kreuz (WMDE)) [18:47:04] fr-tech: i'll start drafting a task to track the changes for that template. we'll want to take a little to make sure it works properly. [18:47:28] 10Fundraising Tech - Chaos Crew: Fraud Attack - https://phabricator.wikimedia.org/T321608 (10Dylsss) According to #acl_WMF-FR, the form https://phabricator.wikimedia.org/maniphest/task/edit/form/16/ is used to create a private fundraising task. [18:55:17] (03Merged) 10jenkins-bot: Make use of the ?? syntax where it makes sense [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/836802 (owner: 10Thiemo Kreuz (WMDE)) [18:55:28] 10Fundraising-Backlog, 10fundraising-tech-ops: Adjust smashpig main.yaml template in puppet to support prod/dev separation - https://phabricator.wikimedia.org/T321729 (10Dwisehaupt) [19:50:06] hey damilare how did you get on today with your damaged search? [20:03:15] (03PS1) 10Ejegg: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/849650 [20:03:21] (03CR) 10Ejegg: [C: 03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/849650 (owner: 10Ejegg) [20:08:54] 10Fundraising Sprint Turtles that are robotic that destroy the whole world with their foot theory, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: New custom Activities Created in Civi - https://phabricator.wikimedia.org/T318529 (10Ejegg) [20:12:36] (03PS5) 10Ejegg: WIP add donatewiki service [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/841965 (https://phabricator.wikimedia.org/T317498) [20:15:19] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/849650 (owner: 10Ejegg) [20:21:15] 10Fundraising-Backlog: AMEX needs to be established as it's own gateway in Civi - https://phabricator.wikimedia.org/T321611 (10XenoRyet) [20:31:55] oh jeez, that'll be complicated [20:33:02] T321611 , that is [20:33:03] T321611: AMEX needs to be established as it's own gateway in Civi - https://phabricator.wikimedia.org/T321611 [20:37:52] 10Fundraising-Backlog: AMEX needs to be established as it's own gateway in Civi - https://phabricator.wikimedia.org/T321611 (10Ejegg) Could we maybe just change this in the gateway reconciliation report? The gateway field is used for a bunch of different things, including determining whether we can send automati... [21:34:05] 10Fundraising Sprint Turtles that are robotic that destroy the whole world with their foot theory, 10Fundraising Tech - Chaos Crew, 10Wikimedia-Fundraising-CiviCRM: Clean up our five character locale data.... - https://phabricator.wikimedia.org/T321251 (10AnnWF) till Oct 26, 2022 civi have | preferred_langua... [21:44:05] fr-tech anyone that worked on the unit tests for the pending transaction resolver, should I just add gateway txn id to the main setup block or have it be adyen specific in the tests? Ingenico will be using the session id right? [21:44:39] hmm dunnooooo [21:44:57] cstone maybe have it be adyen specific? [21:48:41] classic don't think I changed anything yet all the tests pass now [21:52:50] man the time limit before the pending transaction resolver will look at the message always gets me hah [21:53:05] even when I hardcoded it to a minute [21:55:34] (03PS2) 10Cstone: Add check gateway_txn_id to Adyen in the Pending Resolver [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849139 (https://phabricator.wikimedia.org/T321602) [21:55:37] I think we have the unit test at drupal/sites/default/civicrm/extensions/wmf-civicrm/tests/phpunit/Civi/Api4/Action/PendingTransaction/ResolveTest.php [21:56:34] oh nevermind you did the adyen specific one~ [21:56:45] hah yeah thanks everyone for the answers [22:00:17] (03PS3) 10Cstone: Add check for gateway_txn_id to Adyen in the Pending Resolver [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849139 (https://phabricator.wikimedia.org/T321602) [22:00:24] (03CR) 10CI reject: [V: 04-1] Add check for gateway_txn_id to Adyen in the Pending Resolver [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849139 (https://phabricator.wikimedia.org/T321602) (owner: 10Cstone) [22:01:00] (03PS4) 10Cstone: Add check for gateway_txn_id to Adyen in the Pending Resolver [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849139 (https://phabricator.wikimedia.org/T321602) [22:01:25] (03CR) 10CI reject: [V: 04-1] Add check for gateway_txn_id to Adyen in the Pending Resolver [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849139 (https://phabricator.wikimedia.org/T321602) (owner: 10Cstone) [22:01:37] what happened here I just edited a comment haha [22:03:38] (03PS5) 10Cstone: Add check for gateway_txn_id to Adyen in the Pending Resolver [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849139 (https://phabricator.wikimedia.org/T321602) [22:03:44] (03CR) 10jenkins-bot: Add check for gateway_txn_id to Adyen in the Pending Resolver [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849139 (https://phabricator.wikimedia.org/T321602) (owner: 10Cstone) [22:04:16] uhhh anyone know what I broke here fr-tech ^ ? [22:05:26] hmm, says it wants a rebase [22:05:40] but also that it's up to date with master???? [22:05:57] I rebased it twice haha [22:06:02] gut [22:06:04] guh [22:07:06] (03CR) 10Ejegg: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849139 (https://phabricator.wikimedia.org/T321602) (owner: 10Cstone) [22:07:13] (03CR) 10CI reject: [V: 04-1] Add check for gateway_txn_id to Adyen in the Pending Resolver [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849139 (https://phabricator.wikimedia.org/T321602) (owner: 10Cstone) [22:07:17] :P [22:07:18] cursed [22:07:34] should never have updated the comment :P [22:10:18] (03PS1) 10Cstone: WIP: Test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849665 [22:10:25] (03CR) 10CI reject: [V: 04-1] WIP: Test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849665 (owner: 10Cstone) [22:10:28] gotta head out for a bit. see you all later! [22:10:49] wfan: was this like the issue you ran into before with CI? [22:11:59] nope, mine's not merge failed, this CI error looks weird, maybe you can create a small change to test the pr first [22:12:38] yeah thats what the last one was [22:12:44] https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/849665/ [22:12:56] oh rebase issue, my solution is checkout the master and redo the change, then use the same commit number [22:13:20] yeah but this is a completely separate patch [22:13:23] ha seems like the test patch is not working too [22:13:41] ill take a break maybe its just being weird for a bit [22:14:35] sure, if still looks the same, then need to create a ticket for releng like last time haha [22:15:12] do you have a link to that one wfan ? [22:16:21] 10Fundraising Sprint Turtles that are robotic that destroy the whole world with their foot theory, 10Fundraising Tech - Chaos Crew, 10Wikimedia-Fundraising-CiviCRM: Clean up our five character locale data.... - https://phabricator.wikimedia.org/T321251 (10AnnWF) for spanish, civi supports es_ES, es_MX, and e... [22:18:35] let me check [22:20:16] cstone: https://phabricator.wikimedia.org/T317928 [22:21:32] seems like they did not change the code but they help with another commit, and once that pass, mine's pass as well, but I think I tag the right group for this ticket [22:22:32] thanks! [22:22:38] np [22:23:18] hah your code was pending transaction resolver too! [22:23:27] hahaha right [22:23:42] CI doesn't want us to resolve these transactions :P [22:26:49] (03PS1) 10Cstone: WIP: Test 2 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849698 [22:26:55] (03CR) 10CI reject: [V: 04-1] WIP: Test 2 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849698 (owner: 10Cstone) [22:27:02] hmm oh well [22:38:53] (03CR) 10Ahmon Dancy: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849139 (https://phabricator.wikimedia.org/T321602) (owner: 10Cstone) [22:40:54] (03PS1) 10Umherirrender: Build: Use new unit-disallowed-list rule [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/849705 (https://phabricator.wikimedia.org/T279431) [22:41:52] (03Abandoned) 10Cstone: WIP: Test 2 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849698 (owner: 10Cstone) [22:42:07] (03Abandoned) 10Cstone: WIP: Test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/849665 (owner: 10Cstone) [22:42:14] 10Fundraising Sprint Turtles that are robotic that destroy the whole world with their foot theory, 10Fundraising Tech - Chaos Crew, 10Wikimedia-Fundraising-CiviCRM: Clean up our five character locale data.... - https://phabricator.wikimedia.org/T321251 (10AnnWF) for french, civi supports fr_FR, fr_CA, while... [23:11:03] 10Fundraising Sprint Turtles that are robotic that destroy the whole world with their foot theory, 10Fundraising Tech - Chaos Crew, 10Wikimedia-Fundraising-CiviCRM: Clean up our five character locale data.... - https://phabricator.wikimedia.org/T321251 (10AnnWF) for chinese, civi supports zh_TW, zh_CN, while... [23:20:30] can someone review the update_language.yaml at process control, and if looks ok, turn the job on~ Thanks [23:27:12] syntax looks good and the ugly string properly expands to valid json. :) [23:27:42] would it really intend to be run 3 times an hour? [23:27:59] once the schedule is uncommented that is. [23:29:11] sorry, not 3 times an hour, but every 3rd minute [23:30:40] That’s what we did for clean the en to en_US, maybe longer like 10 times per hour? [23:31:48] yeah, if you have a timeout of 5 mins, seems odd to have it run more often than that. [23:32:47] i'm a fan of setting the schedule to what you would expect it to be even if commented out. then there won't be surprises if someone else removed the comment. [23:39:06] Ok will do the adjustment and turn the job on to see what’s going on thanks