You are viewing a single comment's thread from:

RE: [Bug Fix - Merged and Live!] Finally, Busy can edit posts older than 7 days!

in #utopian-io5 years ago

Hi @blockchainstudio, thanks for making this contribution and for submitting a pull request that rectified the error.

Its a great find, prior to this report I have not used the edit feature of the busy.org frontend on a post older than 7 days. I am glad to know it now works. Thanks again.

  1. I like the precise and simple manner in which you described the expected and actual behavior.

  2. The steps were missing in the 'how to reproduce section'. Even though the behavior can be reproduced in as many browsers, it is still required that you state the steps that should be taken to replicate this, be it just two or three steps.

  3. Though you fixed the bug, you failed to analyse what may have caused it. This is unlike your other reports. The inclusion of this technical bit would have further increased your score.

Thanks again. I hope this feedback helps in your future reports :)

Merry Christmas

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? Write a ticket on https://support.utopian.io/.
Chat with us on Discord.
[utopian-moderator]

Sort:  

Thank you for your review.

  1. Recording Of The Bug section actually shows how to reproduce it.

  2. I didn't fail to analyze the cause :) It's too obvious to people who can read the code that fixed, so I thought it even needed not to be explained. I've added the explanation for those who're not familiar with the code. Basically it's because they didn't show edit button for posts older than 7 days. It's that simple :)

Thank you.

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

Coin Marketplace

STEEM 0.28
TRX 0.11
JST 0.031
BTC 69371.89
ETH 3768.85
USDT 1.00
SBD 3.71