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

Extend comment email notification API #973

Merged
merged 13 commits into from Feb 9, 2018

Conversation

voidxnull
Copy link
Member

@voidxnull voidxnull commented Jan 8, 2018

Server-side for #937

Notable changes:

  • Refactored email templates to be more modular.
  • Added daily and weekly tasks for delivering comment notification.
  • Replaced the mute_all_posts user option with comment_notifications that can be set to: 'on', 'off', 'daily', 'weekly'.
  • The updateUser controller was modified to accept the new option (more.comment_notifications).
  • Added email previews (/api/v1/email-template/file_name.ejs).
  • Refactored email tasks to use cached templates.
  • Set up bunyan logging for tasks.

@voidxnull voidxnull changed the title [WIP] Extend comment notification API [WIP] Extend comment email notification API Jan 8, 2018
@coveralls
Copy link

coveralls commented Jan 8, 2018

Coverage Status

Coverage decreased (-0.2%) to 59.348% when pulling d676b50 on feature/comment-notifications-api-ext into d947227 on stable.

@voidxnull voidxnull force-pushed the feature/comment-notifications-api-ext branch from 030147f to 9f6ad38 Compare January 14, 2018 23:54
@voidxnull voidxnull changed the title [WIP] Extend comment email notification API Extend comment email notification API Jan 14, 2018
@voidxnull voidxnull force-pushed the feature/comment-notifications-api-ext branch from 9f6ad38 to d676b50 Compare January 23, 2018 18:03
@voidxnull voidxnull merged commit 229a82f into stable Feb 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants