[02:00:09] (03PS7) 10Ejegg: Adyen Checkout: Apple Pay [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/713551 (https://phabricator.wikimedia.org/T282596) (owner: 10Jgleeson) [02:00:11] (03PS1) 10Ejegg: Apple Pay: correctly format amount for all currencies [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/714876 (https://phabricator.wikimedia.org/T282596) [02:02:58] (03CR) 10jerkins-bot: [V: 04-1] Apple Pay: correctly format amount for all currencies [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/714876 (https://phabricator.wikimedia.org/T282596) (owner: 10Ejegg) [02:04:30] (03PS2) 10Ejegg: Apple Pay: correctly format amount for all currencies [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/714876 (https://phabricator.wikimedia.org/T282596) [02:08:22] sorry eileen, I'll have to get to that review tomorrow [02:17:22] (03PS1) 10Ejegg: Add links for Adyen / Apple Pay [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/714877 [06:07:43] 10WMSE-Content-partnerships-support-2021-Fundraising: Campaign tools in Matomo - https://phabricator.wikimedia.org/T238813 (10Sebastian_Berlin-WMSE) [12:05:17] 10Fundraising-Backlog, 10fundraising-tech-ops: Remove old/no longer used tables in pgehres - https://phabricator.wikimedia.org/T288973 (10Jgreen) [12:07:11] 10Fundraising-Backlog, 10fundraising-tech-ops: Remove old/no longer used tables in pgehres - https://phabricator.wikimedia.org/T288973 (10Jgreen) p:05Triage→03Medium [12:11:35] 10Fundraising-Backlog, 10fundraising-tech-ops: Remove old/no longer used tables in pgehres - https://phabricator.wikimedia.org/T288973 (10Jgreen) [12:15:18] 10Fundraising-Backlog, 10fundraising-tech-ops: Remove old/no longer used tables in pgehres - https://phabricator.wikimedia.org/T288973 (10Jgreen) [12:17:15] 10Fundraising-Backlog, 10fundraising-tech-ops: Remove old/no longer used tables in pgehres - https://phabricator.wikimedia.org/T288973 (10Jgreen) [12:19:53] 10Fundraising-Backlog, 10fundraising-tech-ops: Remove old/no longer used tables in pgehres - https://phabricator.wikimedia.org/T288973 (10Jgreen) 05Open→03Resolved a:03Jgreen [15:18:10] fr-tech does someone else want to try the MG file update script? [15:19:21] sure ejegg [15:20:32] cstone https://www.mediawiki.org/wiki/Fundraising_tech/Deployment#Matching_gifts_employers_list [15:20:48] let me know if those instructions are lacking anything [15:21:19] ok cool [15:23:49] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10fundraising Sprint NULL calorie food cart, 10fundraising sprint Parallel processing roller coaster, and 4 others: Recurring donors segmentation criteria Civi/Acoustic - https://phabricator.wikimedia.org/T283798 (10KHaggard) Thanks @Eileenmcnaughton... [15:28:11] worked nicely ejegg [15:35:53] cool cstone [15:36:53] fr-tech, i'm going to do the 'risky' backup-dc-deploy with the main payments project, temporarily taking out the old-adyen line from LocalSettings and just deploying to frpm2xxx [15:37:10] so please nobody deploy payments-wiki in the next 10 minutes [15:40:39] ok, that's deployed and the localsettings repo is back to pointing adyen at the old integration [15:50:51] dwisehaupt: looks like we'll need to update the CSP headers on payments to get new-adyen working [15:51:03] lemme see if I can find it in puppet [15:52:29] oh hmm, are we doing that through localsettings? [15:53:07] oh nvm, was grepping for underscores instead of dashes [15:55:30] makes sense. [15:56:51] huh, but I can't quite understand why the CSP header is blocking the script load [15:56:56] lemme see if amazon is working [15:59:37] ok, Special:AmazonGateway has the same CSP header on our side and it's able to load scripts from amazon [15:59:41] any meta tags? [16:00:11] nope, not that would change CSP [16:01:34] oooh, it's the data: url [16:01:38] Content Security Policy: The page’s settings blocked the loading of a resource at data: [16:01:45] ("default-src") [16:05:31] ejegg, you available for standup? [16:05:39] oops, be right there [16:05:57] No worries [16:23:21] ejegg: yeah, that is controlled in puppet. we can update as need be. i'm not sure what we will need there. [16:24:40] if there is something in the adyen docs, that would be great to point us at. [16:30:05] ejegg: the failmail zoo is really helpful with the check forget me but then im failing at logging into acoustic haha [16:31:07] arrr [17:35:00] fr-tech I'm going to do another 'secret prod' deploy for payments-wiki - please don't deploy to ALL: for the next ten minutes [17:37:42] ok, reverted the new integration pointer [17:37:46] safe to deploy again [17:38:29] woot, seeing the checkout fields! [17:40:49] and... it seems to have worked [17:40:58] looking in the console [17:42:01] hmm [17:42:45] oh hah, not the test console [17:44:27] hmm, is there lag? [17:46:27] ok, i got the TY email [17:47:29] and it looks like we DID get the auth [17:47:57] IPN message [17:54:06] and it looks like we DID try to capture twice [17:57:03] hmm, still shows as pending on my credit card statement [17:57:58] though we did get an IPN for a successful capture too [17:59:25] hmm, seems we got that a few minutes after the smashpig job did the capture [17:59:49] we're not logging the front-end API calls - maybe we should add more logging there [18:14:41] ejegg|food: woot congrats on that!!!! [19:52:32] 10Fundraising-Backlog, 10fundraising-tech-ops: Only retain fredge data for 2 years - https://phabricator.wikimedia.org/T288971 (10Jgreen) At first glance it looks like we could drop rows from payments_fraud and payments_initial based on date, and then delete rows from payments_fraud_breakdown based on a join.... [20:16:02] 10Fundraising-Backlog, 10fundraising sprint Parallel processing roller coaster, 10fr-donorservices: Civi email address format exception request - https://phabricator.wikimedia.org/T288644 (10SHust) @Eileenmcnaughton - Awesome fun fact, thanks for sharing! [20:31:16] 10Fundraising-Backlog, 10FR-Ingenico: Swedish donations stuck at status 600 - https://phabricator.wikimedia.org/T246710 (10EMartin) Manavi, Mark, Carmen, We are rolling along preparing to deploy our integration over the next few days. We don't need a call tomorrow, but let's please keep the standing meeting... [20:31:20] 10Fundraising-Backlog, 10FR-Ingenico: Swedish donations stuck at status 600 - https://phabricator.wikimedia.org/T246710 (10EMartin) This event has been canceled. Title: Adyen : Wikimedia Placeholder for technical call When: Fri Aug 27, 2021 6am – 6:30am Pacific Time - Los Angeles Joining info: Join with... [20:36:10] 10fundraising-tech-ops, 10DC-Ops, 10ops-eqiad: (Need By: TBD) rack/setup/install - https://phabricator.wikimedia.org/T289812 (10RobH) [20:36:30] 10fundraising-tech-ops, 10DC-Ops, 10ops-eqiad: (Need By: TBD) rack/setup/install - https://phabricator.wikimedia.org/T289812 (10RobH) [20:36:52] 10fundraising-tech-ops, 10DC-Ops, 10ops-eqiad: (Need By: TBD) rack/setup (4) fundraising hosts - https://phabricator.wikimedia.org/T289812 (10RobH) [20:37:40] 10fundraising-tech-ops, 10DC-Ops, 10ops-eqiad: (Need By: TBD) rack/setup (4) fundraising hosts - https://phabricator.wikimedia.org/T289812 (10RobH) [20:38:15] 10fundraising-tech-ops, 10DC-Ops, 10ops-eqiad: Q1:(Need By: TBD) rack/setup (4) fundraising hosts - https://phabricator.wikimedia.org/T289812 (10RobH) a:03Jclark-ctr [20:38:21] 10fundraising-tech-ops, 10DC-Ops, 10ops-eqiad: Q1:(Need By: TBD) rack/setup (4) fundraising hosts - https://phabricator.wikimedia.org/T289812 (10RobH) [21:23:31] why the hell is there zero sample code anywhere online for Apple Pay JS? [21:24:14] seriously searching google or github for anyone actually calling completePaymentMethodSelection with an ApplePayPaymentMethodUpdate gives nothing [21:24:29] jack asked them that and they had like a non answer [21:24:36] of why there wasnt example code [21:24:50] just stuff like the adyen code which passes through whatever the dev sends it [21:26:03] and the Safari console is not spitting out anything useful when i try to eval random things; [21:28:49] woo, let's try what this person did: https://github.com/sumitvekariya/angular-apple-pay/blob/ac5251f8ef0ab1458292dcd865557e8e7b0157b6/src/app/apple-pay/apple-pay.component.ts [21:32:50] that fails [21:40:05] ejegg: is there some setting i need to set with the updated applepay code you did? [21:40:46] cstone no extra settings, no [21:41:08] i went to apple and its says Configuration key 'payment-provider/apple/class' does not exist [21:41:30] cstone oh, you need the updated SmashPig patch in vendor too [21:41:37] i thought i had but lets see [21:41:49] oh nope still on master hah thanks [21:41:50] ooh, did I forget to check in the config.yaml change? [21:41:53] ah, cool [21:42:02] no i was looking right at the yaml change in the code haha [21:42:36] ok cool [22:03:10] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10fundraising Sprint NULL calorie food cart, 10fundraising sprint Parallel processing roller coaster, and 4 others: Recurring donors segmentation criteria Civi/Acoustic - https://phabricator.wikimedia.org/T283798 (10Eileenmcnaughton) @KHaggard do you... [22:13:54] uggh, the onAuthorize call where we can actually see the contact info is called AFTER the onSubmit callback where we post back to our site [22:13:57] booo [22:14:25] so I guess Apple Pay needs to short-circuit the onSubmit [22:14:37] gross [22:16:25] ughhhhh more [22:16:38] we still need the applePayToken from onSubmit [22:16:45] I guess stash that at a higher level? [22:17:53] so... there is a fancy way to use promises to wait until both of those callbacks have happened, in whichever order [22:18:20] do I feel like getting that to work right now? [22:23:07] oho, maybe we CAN get the token in the authorized event [22:23:55] grr, no, adyen JS seems to obfuscate it further [22:36:08] (03PS2) 10Ejegg: Add links for Adyen / Apple Pay [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/714877 [22:41:24] 10Fundraising-Backlog, 10fundraising sprint Parallel processing roller coaster, 10fundraising sprint onion pit, 10FR-Adyen: Adyen Checkout: Pull donor contact from Apple Pay sheet - https://phabricator.wikimedia.org/T285722 (10Ejegg) a:03Ejegg [22:42:22] (03PS1) 10Ejegg: WIP get contact info from Apple Pay sheet [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/715135 (https://phabricator.wikimedia.org/T285722) [22:43:14] also not exactly fun in Safari to try to debug JS events that happen while the Apple Pay sheet has modal'ed the whole application [22:45:10] (03CR) 10jerkins-bot: [V: 04-1] WIP get contact info from Apple Pay sheet [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/715135 (https://phabricator.wikimedia.org/T285722) (owner: 10Ejegg) [22:46:38] (03PS2) 10Ejegg: WIP get contact info from Apple Pay sheet [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/715135 (https://phabricator.wikimedia.org/T285722) [23:00:06] fr-tech should we put debug=true on all the payment links from the dev-payments Main_Page? [23:00:30] I like this idea ejegg [23:00:52] i'm definitely adding it whenever I do apple pay [23:02:42] I have it on all my local links cause [23:02:49] I always forget about it [23:05:53] (03PS1) 10Ejegg: Add debug=true to all payments links [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/715139 [23:08:14] shoot, i can't seem to set up a card with no billing address [23:08:33] I added one for the first card [23:08:43] but now I want to see what happens when you don't have that set up [23:09:04] and if I add another test card it just defaults to the same billing address [23:09:08] and doesn't let me remove it [23:09:36] cstone: did you set up a billing address with your test card? I forget if it lets you finish adding a card without one. [23:10:02] Hmm I am not sure I don't remember adding one [23:10:15] Does it show up in the wallet [23:10:31] yep, in the wallet under the detauls of the selected card [23:10:39] PROBLEM - Host frdb2002 is DOWN: PING CRITICAL - Packet loss = 100% [23:10:41] so plz don't set one up yet if you don't have one [23:10:51] Ok yeah I don't have one [23:11:01] i'm very curious to see what you get when you try this patch: https://gerrit.wikimedia.org/r/715135 [23:11:19] that frdb2002 alert is me doing a reboot after updates. [23:11:24] so the way you get the donor address to show up is to say postalAddress is required [23:11:32] thanks dwisehaupt [23:12:01] Ok lemme try that patch [23:13:03] ACKNOWLEDGEMENT - Host frdb2002 is DOWN: PING CRITICAL - Packet loss = 100% Dwisehaupt reboot for updates [23:15:48] ok so BILLING ADDRESS INVALID in the wallet [23:15:58] when that pops up then just stops there [23:16:06] ooohhhh [23:16:09] that's gross [23:16:23] i have a screenshot if you want to see [23:16:54] thanks! [23:17:43] i just put it on the phab https://phabricator.wikimedia.org/T285722 [23:18:06] 10Fundraising-Backlog, 10fundraising sprint Parallel processing roller coaster, 10fundraising sprint onion pit, 10FR-Adyen, 10Patch-For-Review: Adyen Checkout: Pull donor contact from Apple Pay sheet - https://phabricator.wikimedia.org/T285722 (10Cstone) {F34621217} With no billing address set [23:18:57] cstone: that's common. i've seen that when using applepay before. you are supposed to select that and it'll provide a dropdown for you to choose an address. [23:18:58] hmmmm, it looks pretty dubious that we'd be able to mark those fields optional [23:19:14] it's a bit ugly but as far as i know, the regular behavior. [23:19:26] ah i see in the dropdown [23:20:00] for store purchases, it can be useful, as they can specify a shipping address field that works in the same way, so you can change the shipping address there. [23:20:10] 10Fundraising-Backlog, 10fundraising sprint Parallel processing roller coaster, 10fundraising sprint onion pit, 10FR-Adyen, 10Patch-For-Review: Adyen Checkout: Pull donor contact from Apple Pay sheet - https://phabricator.wikimedia.org/T285722 (10Cstone) {F34621220} Clicking the up down arrow next to th... [23:20:24] dwisehaupt: so I think we would LIKE to be able to charge them even without them telling us their billing address. But the only way to have the widget supply us the addy is to mark it required [23:20:29] (way to many ,s and not enough .s in that statment. :) [23:20:57] ejegg: oh. hmmm... yeah i can see that. [23:21:37] may be a question for adyen since i have no idea. [23:21:56] hah, i doubt they really know that much about the apple pay JS [23:24:23] gonna bike around a few min to clear my head [23:40:59] RECOVERY - Host frdb2002 is UP: PING OK - Packet loss = 0%, RTA = 32.46 ms