Skip to content

Updated Popular Protips emailer with New Relic promotion option #189

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Sep 2, 2014
Merged

Updated Popular Protips emailer with New Relic promotion option #189

merged 9 commits into from
Sep 2, 2014

Conversation

just3ws
Copy link
Contributor

@just3ws just3ws commented Aug 29, 2014

No description provided.

Generated a protips mailer

WIP: query for popular protips directly and bypass Tire and our search implementation

Added mail_view

WIP: wiring up the popular protips

WIP: cleaning up the old weekly digest so I can merge the styles and structure

WIP: formatting a popular protip email

Re-re-add MailView

Pretty close to complete on the popular protips email
@@ -0,0 +1,3 @@
ProtipMailer#popular_protips
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is this file doing here ?

@@ -11,6 +11,10 @@ Here is how you can get started:
* Check out the trending pro tips and share your own. <%= link_to('trending', 'https://coderwall.com/trending') %>
* Display your achievements on your personal website or blog using our javascript badge: <%= api_url %>

Link free swag?

<a href="https://coderwall.com/team/new-relic">Test drive New Relic for free and get a Coderwall tee</a>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tee ? this is not twitter. we can write full words

just3ws added a commit that referenced this pull request Sep 2, 2014
Updated Popular Protips emailer with New Relic promotion option
@just3ws just3ws merged commit 0810d36 into coderwall:master Sep 2, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants