You are viewing a single comment's thread from:
RE: New Project: DailyBrief a WordPress plugin to extend SteemPress
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.
- Nice to see some tests here. You might want to remove the template
test-sample.php
and some other$this->assertTrue(true)
. - You might want to use
$this->pass();
or$this->fail()
to replace$this->assertTrue(true)
and$this->assertTrue(false)
respectively. - I notice that some scripts are for Linux OS, you might want to provide the Windows alternatives (batch) for platform portability
svn-push
andpre-commit
should not be in this code repro.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.
Thank you for your review, @justyy! Keep up the good work!