wpsharks / comment-mail-kb

The Comment Mail™ and Comment Mail™ Pro Knowledge Base
http://comment-mail.com
0 stars 1 forks source link

Can users modify their existing subscriptions? #40

Closed renzms closed 7 years ago

renzms commented 7 years ago

KB Article Creation Checklist

Additional TODO list items go here.


KB Article Published @ comment-mail.com
:page_with_curl: See: Can users modify their existing subscriptions?

:octocat: View Markdown File | :pencil2: Edit Markdown File


renzms commented 7 years ago

@raamdev Please check thanks!

raamdev commented 7 years ago

Looks good! The screenshots only show examples of "Add New Subscription" but the question asks about modifying a subscription. Could you also include a screenshot that shows a subscription being modified?

renzms commented 7 years ago

@raamdev

Please check thanks!

raamdev commented 7 years ago

@renzms Looks a lot better! A bit of feedback:

2017-05-25_18-42-09

renzms commented 7 years ago

@raamdev

Changed:

Ready for check, thanks!

raamdev commented 7 years ago

Looks much better! Please see my notes on the screenshots here: https://github.com/websharks/comment-mail-kb/issues/31#issuecomment-309596113

renzms commented 7 years ago

@raamdev

Updated screens, thanks!

raamdev commented 7 years ago

by accessing their My Comment Subscription Page via the link found at the bottom

Bottom of where?

There they can customize their subscriptions from

Reiterate where "There" is (e.g., "From the My Comment Subscriptions page, they can customize...")

Choosing Post

Those headers above the screenshots are not necessary here, since the screenshots all contain the same headers and since there's no text in between the screenshots. The headers just look redundant.


Also, from now on, please add toc-enable: off to the YAML markup at the top. That will disable the Table of Contents generation on the KB Article when it's published. If there aren't multiple headers or if the KB article is really short (e.g., just 1 or 2 headers), then it doesn't make sense to unnecessarily generate a TOC.

renzms commented 7 years ago

@raamdev

Updated with corrections, please check. Thanks! :)