Would be nice to have to be able to requeue messages for example.

CommentFileSizeAuthor
#5 hook_for_message-2711305-5.patch4.3 KBzaporylie
#4 hook_for_failed_message-2711305-4.patch2.7 KBzaporylie
#3 hook_for_failed_message-2711305-3.patch1.73 KBeiriksm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

eiriksm created an issue. See original summary.

zaporylie’s picture

zaporylie
he/him
Primary language Polish
CreditAttribution: zaporylie at Ny Media AS commented
Issue tags: +7.x-1.0 release blocker
eiriksm’s picture

eiriksm
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: eiriksm at Ny Media AS commented
Status: Active » Needs review
FileSize
hook_for_failed_message-2711305-3.patch1.73 KB
zaporylie’s picture

zaporylie
he/him
Primary language Polish
CreditAttribution: zaporylie at Ny Media AS commented
Title: hook for failed message sending » hook for message success and error
FileSize
hook_for_failed_message-2711305-4.patch2.7 KB

I was actually thinking about message wrapper class passed to sparkpost_mailsend_error and sparkpost_mailsend_success hooks. And to sparkpost_mailsend itself. That would make it easy to i.e resend message. What do you think?

zaporylie’s picture

zaporylie
he/him
Primary language Polish
CreditAttribution: zaporylie at Ny Media AS commented
FileSize
hook_for_message-2711305-5.patch4.3 KB

Forgot about one file :)

eiriksm’s picture

eiriksm
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: eiriksm at Ny Media AS commented
Status: Needs review » Reviewed & tested by the community

Needs .api.php file, will open another issue for it.

  • zaporylie committed ac71f6c on 7.x-1.x 
    Issue #2711305 by zaporylie, eiriksm: hook for message success and error
zaporylie’s picture

zaporylie
he/him
Primary language Polish
CreditAttribution: zaporylie at Ny Media AS commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 7.x-1.x. I fixed a few typos.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.