Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the WWW-Postmark CPAN distribution.

Report information
The Basics
Id: 84512
Status: resolved
Priority: 0/
Queue: WWW-Postmark
People
Owner: Nobody in particular
Requestors: lxp [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Wishlist
Broken in: 0.4
Fixed in: (no value)

History Show all quoted text
  Mon Apr 08 19:43:15 2013 lxp [...] cpan.org - Ticket created
Subject: return further details on successful send
When Postmark successfully sends mail it returns a JSON structure with further details, as documented here: http://developer.postmarkapp.com/developer-build.html#success-response It would be nice if WWW::Postmark->send() could return these details, as it would enable calling code to e.g. log the sent message ID. Since the WWW-Postmark distribution already requires JSON support, the code could simply return a hash reference of the JSON response. This change would be backwards-compatible since "a true value" is still returned as per the documentation.
  Sun Apr 14 07:32:26 2013 lxp [...] cpan.org - Correspondence added
On Tue Apr 09 09:43:15 2013, LXP wrote: Show quoted text
> When Postmark successfully sends mail it returns a JSON structure with > further details, as documented here: > > http://developer.postmarkapp.com/developer-build.html#success-response > > It would be nice if WWW::Postmark->send() could return these details, > as it would enable calling code to e.g. log the sent message ID. > > Since the WWW-Postmark distribution already requires JSON support, the > code could simply return a hash reference of the JSON response. > This change would be backwards-compatible since "a true value" is > still returned as per the documentation.
I have implemented this functionality and submitted a pull request on GitHub: https://github.com/ido50/WWW-Postmark/pull/2
  Sun Apr 14 07:32:42 2013 lxp [...] cpan.org - Reference to https://github.com/ido50/WWW-Postmark/pull/2 added
  Wed May 01 05:57:45 2013 lxp [...] cpan.org - Subject changed from 'return further details on successful send' to 'patch: return further details on successful send'
  Sun May 05 13:39:40 2013 IDOPEREL [...] cpan.org - Correspondence added
Thanks, integrated in v0.5
  Sun May 05 13:39:40 2013 The RT System itself - Status changed from 'new' to 'open'
  Sun May 05 13:39:41 2013 IDOPEREL [...] cpan.org - Status changed from 'open' to 'resolved'