Skip to content
This repository has been archived by the owner on Aug 2, 2019. It is now read-only.

Rewrite the event comment email. #675

Merged
merged 1 commit into from
Feb 3, 2013

Conversation

akrabat
Copy link
Contributor

@akrabat akrabat commented Jan 5, 2013

Rather than a list of key/value pairs, create an email with a useful subject line and body copy that looks good. Also provide a link to the comments section of the event.

Example subject line:

Joind.in event feedback for Web day

Example body:

A new comment has been added to the Ruby day event by estephens on 5th Jan 2013 at 12:37:

My best camp so far, i liked Ryan's talk and Lukes the most

also IRC was amazing

I think it is nice to have more time for questions
and even a session that is more hands on

View on website: http://joindin.localhost/event/view/212#comments

Rather than a list of key/value pairs, create an email with a useful
subject line and body copy that looks good. Also provide a link to
the comments section of the event.

Example subject line:
Joind.in event feedback for Web day

Example body:
A new comment has been added to the Ruby day event by estephens on 5th Jan 2013 at 12:37:

My best camp so far, i liked Ryan's talk and Lukes the most

also IRC was amazing

I think it is nice to have more time for questions
and even a session that is more hands on

View on website: http://joindin.localhost/event/view/212#comments
@dstockto dstockto merged commit ca1d6db into joindin:master Feb 3, 2013
@dstockto
Copy link
Member

dstockto commented Feb 3, 2013

This looks much better than the previous email. I fixed three codesniff errors in the patch and merged in.

Thank you.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants