citruslime.bank-apple-pay | Stripe Developers | Page 1

buoyant lichen Nov 12, 2021, 3:48 PM

#

Usually this means you don't have a payment card set up correctly in your apple pay wallet

#

You need to have a real card saved in your wallet before this can work

candid echo Nov 12, 2021, 3:51 PM

#

I definitely do. I've used apple pay for years.
I don't think you've read the issue correctly.

#

Standard window = broke
Incognito window = working.

I have 3 valid payment cards which I use regularly within my apple pay wallet.

buoyant lichen Nov 12, 2021, 3:54 PM

#

Incognito mode always returns true by default to avoid exposing personal details, that's expected

#

Private, i mean, for safari

candid echo Nov 12, 2021, 3:55 PM

#

So even if the device doesn't support apple pay, it will say it does?

buoyant lichen Nov 12, 2021, 3:55 PM

#

Not working in non-private, is the indication that the payment card isnt set up completely/correctly

candid echo Nov 12, 2021, 3:56 PM

#

I can't stress this enough.

It is setup correctly.
It is my personal device.
I have used apple pay twice today. Once online, once in store.

buoyant lichen Nov 12, 2021, 3:59 PM

#

Ok cool, does the functional apple pay button appear at the top of this docs page for you?
https://stripe.com/docs/stripe-js/elements/payment-request-button

#

(in a non-private tab)

candid echo Nov 12, 2021, 4:00 PM

#

buoyant lichen Nov 12, 2021, 4:00 PM

#

ok cool, thanks for checking 🙂

candid echo Nov 12, 2021, 4:00 PM

#

Yes

buoyant lichen Nov 12, 2021, 4:01 PM

#

Can you share a public URL for your page where i can inspect this myself?

candid echo Nov 12, 2021, 4:05 PM

#

You can use ridebikes.co.uk

https://www.ridebikes.co.uk/cart/?CartIdToRetrieve=eb821da1-1213-4fb5-ab01-af5f61fdb83c

If you use that link, it'll have a basket containing a "Park Tool".
You can proceed through the checkout using Credit Card or Apple Pay, if you see it.

You can spoof data.

If you want to enter a Postcode you can use United Kingdom and enter "LA12 0AA" and choose "Citrus-Lime Ltd" from the address dropdown.

Once you've confirmed your shipping on the following screen you'll enter the payment stage and you ought to see an apple pay accordion and button to click.

#

To be clear, the payment is LIVE. So do not confirm the payment.

#

I'll also elaborate with some extra information I've been provided which is that, several members of staff are all affected by the same issue, but not all of them. It seems as though the result may be cached.

#

#

That's the stripe.paymentRequest object from my citrusPayPaymentRequest variable in the original code snippet above.

buoyant lichen Nov 12, 2021, 4:20 PM

#

I'm on https://www.ridebikes.co.uk/ssl/secure/#checkout-payment that where i would see it right?

candid echo Nov 12, 2021, 4:20 PM

#

Yes, if you can see the other payment options, that's where you would find apple pay

#

E.g.

buoyant lichen Nov 12, 2021, 4:25 PM

#

Has this worked for you previously, and stopped, or has not yet worked?

candid echo Nov 12, 2021, 4:32 PM

#

Right, so ...

Across our platform, we have about 50 websites currently using StripeJS for Apple Pay transactions.

Most of the sites work for everyone, all the time (private and non-private windows).
Some of the sites only work in private some people.

There's no correlation between those who are experiencing the issue across the sites.

Site A may work perfectly for Person X and not Person Y.
But
Site B may work perfectly for Person Y and not Person Z.

I've ruled out that it's device setup.

It seems as though it may have been that apple pay was unavailable on a site for a period of time, perhaps due to domain verification or something like that, and Person X visited it and the result of their "canMakePayment" request was cached. Meaning that when it did become available, the cached result suggested that it wasn't available for their device.

buoyant lichen Nov 12, 2021, 4:32 PM

#

Can you share your stripe account ID so i can inspect your configuration? like acct_1234

candid echo Nov 12, 2021, 4:33 PM

#

We have a connect account.

#

I will share the Ride Bikes acc.

buoyant lichen Nov 12, 2021, 4:33 PM

#

ok can you share this connected account id?

#

i can't find your Stripe.js initialization snippet in the source of the payment page, can you share that too?

candid echo Nov 12, 2021, 4:34 PM

#

acct_1JcTxn2S2rcmFQqy

#

The entire file ...

var citrusPayParentEl = document.querySelector('.citruspay-wallet');

$(document).ready(function ()
{
    initCitrusPayBasket();
});

function initCitrusPayBasket()
{
    $.getScript("https://js.stripe.com/v3/").done(citrusPaySQKLoaded);
}

function citrusPaySQKLoaded()
{
    var citrusPaySDK = Stripe(citrusPayWalletInit.apiKey, {
        stripeAccount: citrusPayWalletInit.mId
    });

    var citrusPayPaymentRequest = citrusPaySDK.paymentRequest({
        country: ecommerceConfig.DespatchCountry.toUpperCase(),
        currency: ecommerceConfig.LocalCurrencyCode.toLowerCase(),
        total: {
            label: 'Cart total',
            amount: citrusPayWalletInit.total,
        },
        requestPayerName: true,
        requestPayerEmail: true,
    });

    citrusPayPaymentRequest.canMakePayment().then(function (result)
    {
        if (result && result.hasOwnProperty('applePay') && result.applePay === true)
        {
            displayCustomCitrusPayWalletButton('apple');
        }
    });
}

function displayCustomCitrusPayWalletButton(type)
{
    var newCitrusPayWalletButton = document.createElement('a');
    newCitrusPayWalletButton.href = '/ssl/secure';
    newCitrusPayWalletButton.classList.add('col-1', 'pos-rel', 'button', 'bSquare', 'buttonChkOut', 'ctrMgr8-top', 'ctrMgr8-bottom', 'centralise', 'backgroundblacki', 'text-center');
    newCitrusPayWalletButton.innerHTML = '<img src="/ssl/images/cards/raw-' + type + '.png?height=22" border="0" />';
    newCitrusPayWalletButton.dataset.type = 'citruspaywallet-' + type;
    newCitrusPayWalletButton.setAttribute('onclick', 'GoToCheckout($(this), event);');

    citrusPayParentEl.appendChild(newCitrusPayWalletButton);
}

#

(p.s. Don't judge by the use of Jquery, I'm required to use it ... it brings great dishonour to my family :D)

#

For clarity, the Stripe Account ID is held in this object property.
citrusPayWalletInit.mId

buoyant lichen Nov 12, 2021, 4:53 PM

#

Looking at this, bear with me

candid echo Nov 12, 2021, 4:53 PM

#

No worries, I'll be available for a while this evening.

buoyant lichen Nov 12, 2021, 5:00 PM

#

OK i think this might be a quick fix

candid echo Nov 12, 2021, 5:00 PM

#

Those are my favourite kind of fixes.

buoyant lichen Nov 12, 2021, 5:00 PM

#

You don't appear to have this particular domain set up for this connected account for apple pay

#

https://stripe.com/docs/stripe-js/elements/payment-request-button?html-or-react=html#html-js-using-with-connect

#

You have a different domain set up for that account via your platform

#

and every domain must be registered (using the well-known file etc)

candid echo Nov 12, 2021, 5:02 PM

#

With us being a connected account, the domain is registered against the parent account.

#

So it's registered against our account.

#

Each site on our platform has the well known file on it and was registered via the API during onboarding.

buoyant lichen Nov 12, 2021, 5:04 PM

#

RIght, but because you are doing direct charges (initializing stripe.js with the connected account):

var citrusPaySDK = Stripe(citrusPayWalletInit.apiKey, {
        stripeAccount: citrusPayWalletInit.mId
    });

this needs to be registered for that account via your platform

#

this is a somewhat annoying config issue we're trying to improve

candid echo Nov 12, 2021, 5:04 PM

#

The onboarding developers stated that this was the way it needed to be done.

buoyant lichen Nov 12, 2021, 5:05 PM

#

If you were doing destination charges it would be for your platform, yes, but with that stripe account parameter for direct charges you need to register it for the connected account

candid echo Nov 12, 2021, 5:06 PM

#

The payment intent is setup with the Destination property.

buoyant lichen Nov 12, 2021, 5:07 PM

#

oh, you want destination charges?

candid echo Nov 12, 2021, 5:07 PM

#

{
    PaymentIntentCreateOptions paymentOptions = new PaymentIntentCreateOptions()
    {
        CaptureMethod = "automatic",
        Amount = CalculateFinancialValueForStripe(this.paymentDetailModel.GrandTotal),
        Currency = this.processorInfo.ISOCurrency,
        PaymentMethodTypes = new List<string>(),
        ApplicationFeeAmount = CalculateFinancialValueForStripe(fewFromCalculator),
        Metadata = new Dictionary<string, string>()
        {
            {
                "Billing Customer Name",
                this.paymentDetailModel.BillAddress.FirstName + " " + this.paymentDetailModel.BillAddress.LastName
            },
            {
                "Billing Customer Email",
                this.paymentDetailModel.BillAddress.EMail
            }
        },
        TransferData = new PaymentIntentTransferDataOptions()
        {
            Destination = this.processorInfo.MerchantID
        },
        OnBehalfOf = this.processorInfo.MerchantID
    };
}

buoyant lichen Nov 12, 2021, 5:07 PM

#

In that case, remove the stripeAccount parameter from the stripe.js initialization

candid echo Nov 12, 2021, 5:08 PM

#

If I can be explicitly clear.
if I was to pay using the apple pay button, the transaction completes perfectly fine. The payment is taken from our platform account and subsequently transferred from the connected account fine too.

buoyant lichen Nov 12, 2021, 5:09 PM

#

I'm not sure what you mean by "If i was to pay..."

#

I'm saying if you're trying to use destination charges, then the pattern for initializing stripe.js is currently not set up for that. THe use of the stripeAccount optional param is for direct charge configurations

merry dagger Nov 12, 2021, 5:09 PM

#

taking over for @buoyant lichen
If you are using destination charges (which your .NET code is doing) you should not do var citrusPaySDK = Stripe(citrusPayWalletInit.apiKey, { stripeAccount: citrusPayWalletInit.mId }); at all

#

instead you should do var citrusPaySDK = Stripe(citrusPayWalletInit.apiKey);

candid echo Nov 12, 2021, 5:10 PM

#

OK, so if I click on the apple pay button on ride bikes ...
Pay via apple pay ...
The payment intent completes correctly. Debiting monies from the correct locations.

#

I set up the StripeJS calls exactly as instructed by Maxime Locqueville (maximelocqueville@stripe.com)

#

He is one of the partner onboarding developers (or something of that ilk).

candid echo Nov 12, 2021, 5:13 PM

#

merry dagger instead you should do ``` var citrusPaySDK = Stripe(citrusPayWalletInit.apiKe...

I originally did setup the StripeJS library like this, but was told to change it to how it is now.

merry dagger Nov 12, 2021, 5:13 PM

#

@candid echo the person who told you was confused/incorrect, it happens as Connect + Apple Pay is really complex.
You use Destination Charges right now per your .NET code, if that's expected please change the Stripe.js code

#

If you are not supposed to use Destination Charges, then change your .NET code

candid echo Nov 12, 2021, 5:14 PM

#

I appreciate I may be coming off as difficult here, but I'm just trying to sanity check the conflicting information.

candid echo Nov 12, 2021, 5:17 PM

#

merry dagger <@!466576480198066176> the person who told you was confused/incorrect, it happen...

That's cool, I'll definitely give that a try ...

Before I do though, can I just query that if you're saying this is wrong ...
How come it's working perfectly fine for us, in the sense that we've completed 00s of £ and € worth of payment intents via apple pay and the money is being taken from the correct accounts.

merry dagger Nov 12, 2021, 5:18 PM

#

I can tell you 100% that it's impossible it works with the current code you have

candid echo Nov 12, 2021, 5:20 PM

#

This payment intent is from yesterday ...
pi_3Juj84ECYMgji7vm1dzbCJE2

merry dagger Nov 12, 2021, 5:23 PM

#

(removed the pictures as they don't really help)
Looking at that id

candid echo Nov 12, 2021, 5:23 PM

#

I understand

#

Thanks

merry dagger Nov 12, 2021, 5:35 PM

#

Okay so that payment is a destination charge. So it's impossible that your code explicitly passes the connected account id in the Stripe.js initialization

#

Do you have a live URL where I can test this/confirm?

candid echo Nov 12, 2021, 5:37 PM

#

Specifically, for the website related to the payment intent you just inspected ...

#

https://www.ridebikes.co.uk/cart/?CartIdToRetrieve=5b2079a1-8e03-4a17-8cd5-9ca1c2891186

#

You can use that link, I created a shareable cart for you.

merry dagger Nov 12, 2021, 5:44 PM

#

where do I click to find the Apple Pay button or the Google Pay button?

candid echo Nov 12, 2021, 5:49 PM

#

So the Apple Pay button would be down the page in the cart.

#

#

You can use ridebikes.co.uk

https://www.ridebikes.co.uk/cart/?CartIdToRetrieve=eb821da1-1213-4fb5-ab01-af5f61fdb83c

If you use that link, it'll have a basket containing a "Park Tool".
You can proceed through the checkout using Credit Card or Apple Pay, if you see it.

You can spoof data.

If you want to enter a Postcode you can use United Kingdom and enter "LA12 0AA" and choose "Citrus-Lime Ltd" from the address dropdown.

Once you've confirmed your shipping on the following screen you'll enter the payment stage and you ought to see an apple pay accordion and button to click.

merry dagger Nov 12, 2021, 5:50 PM

#

There's no button anywhere for me so I'm struggling to help you unfortunately

candid echo Nov 12, 2021, 5:50 PM

#

That’s my exact point. The button only shows in incognito on some of our sites but not all of them.

#

Some sites it works fine.

#

But it’s mixed results for everyone.

merry dagger Nov 12, 2021, 5:51 PM

#

yeah but I am 100% sure you code just can't work at all

#

the only way it works is to remove that stripeAccount thing

candid echo Nov 12, 2021, 5:52 PM

#

I’ll definitely try it, thanks m just stumped as to why the payments are working fine if the code shouldn’t work at all

merry dagger Nov 12, 2021, 5:52 PM

#

Maybe that's not the real code or the exact page? It's hard to say unfortunately without more context on what you built. The way your code is build, the code would 100% error even if the button appeared

candid echo Nov 12, 2021, 5:52 PM

#

All those transactions in the images I sent are from live sites, running the same code and within the last week or so

#

It’s a globally deployed app

#

So all 500 of our sites run the same code base

#

When we release a new version they’re all updated at the same time

#

I’m telling you 100% that’s the code driving those sales, as hard as it may be to believe.

merry dagger Nov 12, 2021, 5:54 PM

#

Just to clarify: I work for Stripe, I've been here for years, I know our products extensively. The code you shared literally means "please confirm the PaymentIntent on the connected account" it would 100% error with a 404 if the PaymentIntent is on the platform

candid echo Nov 12, 2021, 5:54 PM

#

And I understand.

#

I wrote the code

merry dagger Nov 12, 2021, 5:54 PM

#

The only thing I can think of is a bug where your code will pass your own platform account id in that hash

candid echo Nov 12, 2021, 5:55 PM

#

That must be the case then, since I’d bet very good money on that working.

#

I genuinely do believe you (that our code is wrong) but I also am 1000% confident that the same code is also, somehow working.

#

Im going to test it on Monday morning using your code. Shall I DM you on here with the results?

merry dagger Nov 12, 2021, 5:59 PM

#

no need to DM no but you can circle back in the channel. But yeah okay if you think you might be passing your own account id sometimes it'd explain everything. And if you remove the entire stripeAccount option you should be all set!

candid echo Nov 12, 2021, 6:00 PM

#

No we definitely are not passing our ID.

#

We read the customers account ID.

#

But as I said, I’ll definitely try your code. It makes sense that it would work how you said. My confusion came from it actually working with the customers account.

merry dagger Nov 12, 2021, 6:01 PM

#

sounds good

candid echo Nov 12, 2021, 6:03 PM

#

Thanks for your help!
Do you want me to report back to this thread on Monday or just in the general channel?

merry dagger Nov 12, 2021, 6:04 PM

#

the thread will be archived, but just post in the main channel and mention your thread and we can reopen it (or create a new one)

candid echo Nov 12, 2021, 6:05 PM

#

Cheers fella!

#

Have a great weekend.

#citruslime.bank-apple-pay