Copied to clipboard

Flag this post as spam?

This post will be reported to the moderators as potential spam to be looked at


  • Ben McKean 272 posts 549 karma points
    Aug 26, 2016 @ 16:23
    Ben McKean
    1

    Merchello - Notification on Order Confirmation not being sent

    Hi,

    I've got the following set up for my email notifications but it just doesn't seem to be getting sent. I've checked logs and can't see any errors.

    enter image description here

    enter image description here

    enter image description here

    I tried adding a breakpoint to the OrderCreation view (not sure if this would get hit) I've tried it just with the default text in it (no logic).

    At what point, what event is this email sent?

    Thanks

    Ben

  • Rusty Swayne 1655 posts 4993 karma points c-trib
    Aug 26, 2016 @ 17:37
    Rusty Swayne
    0

    The OrderConfirmation is generally sent after a successful payment - unless you are using the PayPalExpress provider (or some other redirecting provider) which there is an event to handle.

    Are you using the FastTrack starter or are you working on a custom build?

    Also, are you sure you have an email service setup on your local machine?

  • Ben McKean 272 posts 549 karma points
    Aug 26, 2016 @ 17:51
    Ben McKean
    0

    It's a custom build. Using PayPal and a or custom Stripe provider.

    Anyway I can raise the event to trigger it?

  • Rusty Swayne 1655 posts 4993 karma points c-trib
    Aug 26, 2016 @ 19:40
  • Ben McKean 272 posts 549 karma points
    Aug 26, 2016 @ 20:21
    Ben McKean
    0

    I'm using smtp4dev and it doesn't receive anything. No error message from what I can see in the logs.

    Is there anything I should look for in the log?

    For PayPal, I'm using the built in PayPal payment method which redirects to PayPal and then returns to a thank you page. How does it now that the payment has been a success as it doesn't go back to an end point?

    For the custom Stripe provider, should I add anything to it to have the Notification.Trigger?

  • Ben McKean 272 posts 549 karma points
    Aug 26, 2016 @ 20:49
    Ben McKean
    0

    I've tried adding to my Stripe payment provider:

    Notification.Trigger("OrderConfirmation", result.Payment);
    

    and tried:

    Notification.Trigger("OrderConfirmation", result.Payment, new[] { result.Invoice.BillToEmail });
    

    but I get the following error:

    Method not found: 'Void Merchello.Core.Notification.Trigger(System.String, System.Object)'.
    

    I tried :

    Notification.Trigger("OrderConfirmation", result.Payment, new[] { result.Invoice.BillToEmail }, Topic.Notifications); 
    

    but I get build errors saying "No overload for method 'Trigger' takes 4 arguments"

  • Rusty Swayne 1655 posts 4993 karma points c-trib
    Aug 26, 2016 @ 20:53
    Rusty Swayne
    0

    I had a bug in the event handler in 2.1.0 - just fixed in 2.2.0. What version of Merchello are you using?

  • Ben McKean 272 posts 549 karma points
    Aug 26, 2016 @ 20:56
    Ben McKean
    0

    Version 2.1.0

    I'm really near the end of this project so a bit wary to upgrade! is it a pretty minor upgrade?

  • Rusty Swayne 1655 posts 4993 karma points c-trib
    Aug 26, 2016 @ 21:03
    Rusty Swayne
    0

    The main feature addition is the shared product options which does do some db schema changes. The migration should handle it - so you should not need to worry. Always a good idea to backup before an upgrade though =)

    I listed the schema changes in the release notes posted in this forum.

    There were quite a few performance tweaks that you'll benefit from ...

  • Ben McKean 272 posts 549 karma points
    Aug 26, 2016 @ 21:08
    Ben McKean
    1

    OK, I'll have a look on Monday, 10pm here in the UK.

    Good news is I've got the notification working for the Stripe provider now so just the PayPal provider to go with hopefully the upgrade will handle.

  • Rusty Swayne 1655 posts 4993 karma points c-trib
    Aug 26, 2016 @ 22:50
    Rusty Swayne
    0

    I'd love to get a Stripe provider put into the Core - I've been speaking to Alex Lingdren (Flight Path) about his - did you say you have done a custom one as well?

    Hoping to create a branch where we can get it in there ... is this something you'd be interested in helping out with or at least letting us see your implementation?

  • Lee 1130 posts 3088 karma points
    Aug 27, 2016 @ 04:35
    Lee
    0

    Would be awesome if you could share your Stripe provider with us :)

  • Ben McKean 272 posts 549 karma points
    Aug 27, 2016 @ 07:51
    Ben McKean
    0

    Yes, I've spoken to Alex. I've taken inspiration from his but his was for an older version of the Stripe API which was no longer supported so I've replaced his code with code using Stripe.net.

Please Sign in or register to post replies

Write your reply to:

Draft