You are viewing a single comment's thread from:

RE: New Project: DailyBrief a WordPress plugin to extend SteemPress

in #utopian-io5 years ago

Thank you for your contribution @dfroberg ! Interesting and creative project and I am sure this will be highly useful especially for those new steemians who don't have large RC and tend to use steemit like a twitter-like micro blogs.

  1. Nice to see some tests here. You might want to remove the template test-sample.php and some other $this->assertTrue(true).
  2. You might want to use $this->pass(); or $this->fail() to replace $this->assertTrue(true) and $this->assertTrue(false) respectively.
  3. I notice that some scripts are for Linux OS, you might want to provide the Windows alternatives (batch) for platform portability
  4. svn-push and pre-commit should not be in this code repro.
  5. language\dailybrief.pot is empty. You might want to add more localization placeholders

For future versions, I'd like to see the filtering/selection for the posts e.g. number of posts, the tags etc. And most importantly, it would be nice to put this on wordpress.com

Nice work!

Your contribution has been evaluated according to Utopian policies and guidelines, as well as a predefined set of questions pertaining to the category.

To view those questions and the relevant answers related to your post, click here.


Need help? Chat with us on Discord.

[utopian-moderator]

Sort:  

Thank you for your review, @justyy! Keep up the good work!

Coin Marketplace

STEEM 0.19
TRX 0.13
JST 0.028
BTC 64625.84
ETH 3233.39
USDT 1.00
SBD 2.65