Skip to content

Conversation

@adekbadek
Copy link
Member

All Submissions:

Changes proposed in this Pull Request:

Because the order was created as completed, an email was sent right away to the site admin, with a 0 as order total.

How to test the changes in this Pull Request:

  1. Choose "Stripe" as the Reader Revenue platform and make sure your site is available publicly (so Stripe webhook can be called)
  2. On master, make a donation using the Donate block
  3. Observe an email is sent to site admin with no order details and 0 as total
  4. Switch to this branch, observe the email sent has order details and correct amount

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully ran tests with your changes locally?

Because the order was created as completed, an email was sent right away
to the site admin, with a 0 as order total.
@adekbadek adekbadek added the [Status] Needs Review The issue or pull request needs to be reviewed label May 17, 2022
@adekbadek adekbadek requested a review from a team as a code owner May 17, 2022 13:11
@dkoo
Copy link
Contributor

dkoo commented May 19, 2022

@adekbadek How do you ensure that the admin email is sent? I followed the testing instructions but never got an email.

1 similar comment
@dkoo
Copy link
Contributor

dkoo commented May 19, 2022

@adekbadek How do you ensure that the admin email is sent? I followed the testing instructions but never got an email.

@adekbadek
Copy link
Member Author

adekbadek commented May 20, 2022

@dkoo – can you verify if the webhook was executed and returned 200, in Stripe dashboard?

Copy link
Contributor

@dkoo dkoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While testing this, I found that there's a conflict with the Salesforce webhooks that are active when a Salesforce connection is established. That connection should probably only be active if the Reader Revenue platform is Newspack, so I'll submit a fix in a separate PR.

After fixing that I was able to confirm the fix in this PR:

Screen Shot 2022-05-20 at 11 13 25 AM

@github-actions github-actions bot added [Status] Approved The pull request has been reviewed and is ready to merge and removed [Status] Needs Review The issue or pull request needs to be reviewed labels May 20, 2022
@adekbadek adekbadek merged commit 7c30b09 into master May 23, 2022
@adekbadek adekbadek deleted the fix/stripe-woo-orders branch May 23, 2022 10:23
@matticbot
Copy link
Contributor

🎉 This PR is included in version 1.84.0-alpha.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@matticbot
Copy link
Contributor

🎉 This PR is included in version 1.84.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released on @alpha released [Status] Approved The pull request has been reviewed and is ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants