Plots2: Include one-line outreach-oriented note in email footers
Currently, emails look something like (depending on if they're questions, comments, notes, etc):
Hi! There's been a new answer posted for the question 'Can we use a timelapse or video camera to monitor particulate air plumes?' that you liked. Do NOT reply to this email; click this link to see the answer:
https://publiclab.org/questions/gretchengehrke/09-20-2016/question-would-it-be-feasible-to-use-a-stationary-camera-to-conduct-emissions-opacity-evaluation#a105
Here is the answer that gretchengehrke wrote:
It turns out that Alternative Method 82 is pretty strict in its procedure, so timelapse and video cameras wouldn't be able to explicitly do that method, but I think they could probably prompt an inspection. Anyway, I recently learned a lot more about EPA Method 9, and some more about Alternative Method 82, and wrote a blog post about it: https://publiclab.org/notes/gretchengehrke/10-28-2016/colorado-regulations-hotsheet
You can comment on the answer at: https://publiclab.org/questions/gretchengehrke/09-20-2016/question-would-it-be-feasible-to-use-a-stationary-camera-to-conduct-emissions-opacity-evaluation#answer-105-comment
Report abuse to: [email protected]
A typical email (for a new note) is generated here: https://github.com/publiclab/plots2/blob/master/app/views/subscription_mailer/notify_node_creation.html.erb
We could include a line at the bottom using the Features system, using the feature() helper:
<% cache('feature_email-footer') do %> // <== this caches and uses a cached version of the feature, which is expired when the feature is edited
<%= feature('email-footer') %>
<% end %>
Then the actual text could be set using the features system -- available to admins at /features.
jywarren
All 18 comments
@jywarren mind if i take this one?
ykl7
on 31 Oct 2016
That would be great, but we may be waiting on a fix from the library that
would generate this...
On Oct 30, 2016 9:25 PM, "ykl7" [email protected] wrote:
@jywarren https://github.com/jywarren mind if i take this one?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/publiclab/plots2/issues/932#issuecomment-257211598,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABfJ20C3Okv_d1b6wZJ4_Wjf1V8eAdiks5q5W3EgaJpZM4KkdiP
.
jywarren
on 31 Oct 2016
I could wait for the library fix if needed. If it is not resolved, I'll look into it.
ykl7
on 31 Oct 2016
OK -- I'd like to use the library, as manually including names could be
quite onerous every time we publish. Thanks, and if you're looking for a
challenge in the meantime, I've added some new help-wanted issues.
On Sun, Oct 30, 2016 at 9:35 PM, ykl7 [email protected] wrote:
I could wait for the library fix if needed. If it is not resolved, I'll
look into it.—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/publiclab/plots2/issues/932#issuecomment-257212368,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABfJ2Tk_tlSW0QYyOy_T5rcIDKiLxkRks5q5XAigaJpZM4KkdiP
.
jywarren
on 31 Oct 2016
That sounds good
ykl7
on 31 Oct 2016
@ykl7 Still working on it?
ryzokuken
on 4 Nov 2016
@ryzokuken I was told to wait for a library fix by @jywarren . If that fix doesn't work, I'll do it, but no word yet.
ykl7
on 4 Nov 2016
okay, cool.
ryzokuken
on 4 Nov 2016
Hi, all -- the issue, in case you want to take a crack at it, is this one: https://github.com/RichardLitt/name-your-contributors/issues/15
If we can get that to work, I think we can move forward with this!
jywarren
on 4 Nov 2016
Ack, apologies, I'd mixed up two issues!!! #931 depends on name-your contributors, not this. This is ready to be worked on, @ykl7 -- apologies for the mixup! I'm looking at too many issues these days with all of your excellent contributions coming in so fast! :-)
jywarren
on 4 Nov 2016
No worries, @jywarren :grin:
Keep up the good work!
ryzokuken
on 4 Nov 2016
@jywarren I'll try to get this done over the weekend and send a PR as soon as possible
ykl7
on 4 Nov 2016
@jywarren Does this have to be added at the bottom of each email sent or will it be just for a new note creation?
ykl7
on 6 Nov 2016
I think we could do it for all emails, thanks!
On Nov 6, 2016 6:41 AM, "ykl7" [email protected] wrote:
@jywarren https://github.com/jywarren Does this have to be added at the
bottom of each email sent or will it be just for a new note creation?—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/publiclab/plots2/issues/932#issuecomment-258675526,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABfJ-Gl1MIwFZ1BS17oPglUdGAwZZ0mks5q7bzXgaJpZM4KkdiP
.
jywarren
on 8 Nov 2016
Hello -- how's this going? We could definitely start with just one email, to confirm that it works, then open a new issue to build on your first step. How does that sound?
jywarren
on 22 Nov 2016
That sounds good, I was trying to do the same in the last PR I opened, to get one working then work on the rest.
ykl7
on 23 Nov 2016
:-) super rad!
On Wed, Dec 7, 2016 at 2:31 PM, ykl7 notifications@github.com wrote:
Closed #932 https://github.com/publiclab/plots2/issues/932.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/publiclab/plots2/issues/932#event-885779324, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AABfJ0RWIOoktegccISlaTqL8lZ1Vk_5ks5rFwmKgaJpZM4KkdiP
.
jywarren
on 7 Dec 2016
Do you think you'd be interested in this somewhat related issue? #931
jywarren
on 7 Dec 2016
Related issues
RuthNjeri
·
3Comments
jywarren
·
3Comments
ebarry
·
3Comments
jywarren
·
3Comments
grvsachdeva
·
3Comments