send e-mail notification to author if his post gets faved
GoogleCodeExporter opened this issue · comments
If a post gets fave'd, it would be cool to notify the author of that post
via e-mail.
At least I often want to see if my post has gotten faved. But it's kind of
tedious to check this by browsing the site.
Original issue reported on code.google.com by jjhel...@gmail.com on 2 Nov 2007 at 5:19
Assigning 2008-Q1 milestone. This needs to happen in Jan-Mar.
Original comment by jjhel...@gmail.com on 29 Jan 2008 at 7:42
- Added labels: Milestone-2008-Q1
Ok the functionality is now there (r101) but it has been commented. I can't be
arsed to figure out how to set up
the mail server for PHP (not that I even know what my mail server at home is
:)), so someone either test this at
home or on saasta.fi.
The modifications are in /saasta-handlefaves.php lines 21-30.
It would seem that if the outgoing mail server isn't configured, then it
doesn't break anything, the mail just
doesn't get sent.
Original comment by mikko.ur...@gmail.com on 2 Feb 2008 at 2:39
Great! I'll try if my vmware Debian can be arsed to send e-mail.
Original comment by jjhel...@gmail.com on 2 Feb 2008 at 3:08
The fix works, I just tried it on my computer.
I slightly disagree with the implementation though. I'd rather not e-mail the
poster
who faved the post -- we've had an unwritten policy on making faves anonymous
and I'd
like to stick with that policy unless we have a good reason not to.
Anyway, I changed it a bit (See r102) to remove "favor" name from the e-mail &
modified it to use "wp_mail" instead of "mail" which appears to set From address
automatically.
This is how the e-mail looks like now:
To: <jope>
Subject: [saasta.fi] - your post got faved!
From: WordPress <wordpress@192.168.231.129>
Message-ID: <fe9a95f50de68e4ba314e909b65daeb9@192.168.231.129>
X-Priority: 3
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Content-Type: text/plain; charset="UTF-8"
Your saasta 'kuvatesti' (http://localhost/saasta/?p=2472) was just faved! High
five!
* *
Good enough for me. Mikko, works ok for you?
Original comment by jjhel...@gmail.com on 2 Feb 2008 at 3:35
In response to Mikko's e-mail:
> Mos' def! I agree with the anonymity completely.
>
> Hmm, one small improvement might be in order - should we say "your
> saasta now has X faves" or something? You'd just have to do one more
> query per fave:
> SELECT count(*) from saasta_faves where post_id=[post id]
Agreed.
Original comment by jjhel...@gmail.com on 2 Feb 2008 at 4:17
Go Mikko, it's all good now. Closing.
Original comment by jjhel...@gmail.com on 2 Feb 2008 at 6:24
- Changed state: Verified