Re: PATCH: Add "mail me this message" feature to archive viewer

From: Amir Rohan <amir(dot)rohan(at)zoho(dot)com>
To: PostgreSQL www <pgsql-www(at)postgresql(dot)org>
Cc: Stefan Kaltenbrunner <stefan(at)kaltenbrunner(dot)cc>, "magnus(at)hagander(dot)net" <magnus(at)hagander(dot)net>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Stephen Frost <sfrost(at)snowman(dot)net>, Andres Freund <andres(at)anarazel(dot)de>
Subject: Re: PATCH: Add "mail me this message" feature to archive viewer
Date: 2015-10-21 13:51:02
Message-ID: 56279846.7080105@zoho.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-www

On 10/12/2015 04:56 PM, Amir Rohan wrote:
> Hi,
>
> See attached patch implementing this feature, as discussed in the
> "No easy way to join existing discussion when not subscribed" thread
> <56138D2D(dot)1000304(at)zoho(dot)com>.
> As my troublesome mail client turned that thread into a mess, this
> should server as a fresh start.
>
> The patch adds a "via email" link beside "[raw] or [whole thread]"
> in the message view. A Mailqueue implementation was copied verbatim
> from pgweb and so the mail worker at
> `tools/mailqueue/send_queued_mail.py` needs to run periodically to drain
> the queue.
>
>
> It was agreed that restricting this feature to logged-in community users
> was necessary and sufficient, so I have added the necessary decorators
> to the view. Because the archive app isn't currently
> hooked in to the community login system, and because I cannot fix that
> without aid from the web team, they are commented out currently.
>
> Optional patch 0002 uses django's "flash" messages extension to provide
> visual feedback to the user using a very soothing shade of green.
>
> Screenshot: http://bit.ly/1R71P6U
>
> Regards,
> Amir
>

Bump. Is there something more I can do to get this live on the archives?
I've wanted to actually make use of this several times since
implementing it.

Regards,
Amir

In response to

Responses

Browse pgsql-www by date

  From Date Subject
Next Message Amir Rohan 2015-10-21 13:54:04 Re: PATCH: add "current" version link to docs page
Previous Message Amir Rohan 2015-10-21 13:49:00 Re: pgarchives: strip angle brackets when checking for msgid search