solidusio / solidus_bolt

BSD 3-Clause "New" or "Revised" License
1 stars 2 forks source link

Add webhook to capture payments #37

Closed Naokimi closed 2 years ago

Naokimi commented 2 years ago

Fixes https://github.com/nebulab/solidus_bolt/issues/33

Bolt request verification has been temporarily commented out and will be implemented in https://github.com/nebulab/solidus_bolt/issues/79 Fixed in commit 12df15f5b4e4e609ba30463aa8284955cde9009f

Manual testing results: Full capture: https://merchant-sandbox.bolt.com/transaction/B3LQ-97ZW-BLP3 image

Partial capture: https://merchant-sandbox.bolt.com/transaction/W2YD-RLG4-DCP2 image

DanielePalombo commented 2 years ago

Does it work for you?

I've got two main errors:

1 ~ActionView::Template::Error (Missing partial spree/api/payments/source_views/_bolt with {:locale=>[:en], :formats=>[:json], :variants=>[], :handlers=>[:jbuilder]} when the webhook is received~ ref https://github.com/nebulab/solidus_bolt/pull/62/commits/c526472570aafb1932569e20edcf4e6515a9be72 2 I had to skip verify_bolt_request check

A part these, when the webhook is received the payment is captured as expected.

Naokimi commented 2 years ago

tested by calling the controller using Postman and confirmed the code works as expected

DanielePalombo commented 2 years ago

Why you didn't test it with Bolt webhooks?