You are viewing a single comment's thread from:

RE: @ilovecoding - The SteemRebloggerService supports BlackList Tags and Automatic Voting

in #utopian-io6 years ago

Hi there,

I have noticed @ilivecoding in a couple of my posts. Might be a good concept to follow programming related content.

Reveal spoiler

permlink1.startsWith("re-")))

This assumption may work on most cases but it also may create false positives.

This is not the best way to determine is a comment object is a main comment (post) or not. I would recommend using "depth" property. If the depth is 0, that mean's it's a main comment (post), else if the depth is greater than 0, it means it's a reply.

However, getting the depth property requires one extra get_content call. So, there is a tradeoff :)

Also, automatic votes on the posts are a good touch but I couldn't see a minimum check for the voting power. I think it should skip voting once the vp goes below the %80 limit.

One a side note, commit messages should be descriptive. "Update x.html" kind of commit messages don't make much sense to the other people browsing the git history.


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.



Chat with us on Discord.
[utopian-moderator]Need help? Write a ticket on https://support.utopian.io/.

Sort:  

Thank you for your review, @emrebeyler!

So far this week you've reviewed 4 contributions. Keep up the good work!