[Bug Fix - Merged and Live!] Finally, Busy can edit posts older than 7 days!
Project Information
- Repository: https://github.com/busyorg/busy
- Project Name: Busy
Update: Now my pull request is merged and you can actually edit! @sekhmet, thanks a lot for reviewing and merging quick! So if you can't find this bug, it's because I already fixed it :)
This is not just a bug reporting. I've fixed the code and finished testing. Pull request is also made.
Issue: https://github.com/busyorg/busy/issues/2137
Pull request: https://github.com/busyorg/busy/pull/2138
Expected behavior
After HF20, all posts (i.e., older than 7 days) should be editable.
Actual behavior
Busy Still doesn't allow to edit old posts. This is because there is still a code that check the date of the post and doesn't show "edit" button when it's older than 7 days.
How to reproduce
This happens all the time on every platform. (See the "Recording of the bug" below)
Recording Of The Bug
Proof of Test
It worked like a charm :)
Here is my old post that I actually edited by using Busy on my local dev server. Of course, the actual edit has been reflected on the real Steem blockchain. https://steemit.com/@blockchainstudio/steem-python-library-sbdtorshares-bugfix
This is my Christmas gift to Steem community :) Hope that it'll be merged soon. Merry Christmas!
ps. Since I gave you a present, please make Steem official Python library included in the Utopian whitelist (of course, in addition to generous voting :) Then I may fix more bugs :) So many people are using Busy. Enjoy editing!
ps2. Now my pull request is merged and you can actually edit! @sekhmet, thanks a lot for reviewing and merging quick!
GitHub Account
https://github.com/economicstudio
Issue: https://github.com/busyorg/busy/issues/2137
Pull request: https://github.com/busyorg/busy/pull/2138
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.
I like the precise and simple manner in which you described the expected and actual behavior.
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.
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]
Thank you for your review.
Recording Of The Bug section actually shows how to reproduce it.
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!
Great work! You have increased the convenience of "busy" users. :)
Thanks a lot, now it's live. You can edit :)
곰돌이가 @lostmine27님의 소중한 댓글에 $0.017을 보팅해서 $0.006을 살려드리고 가요. 곰돌이가 지금까지 총 2224번 $28.176을 보팅해서 $27.659을 구했습니다. @gomdory 곰도뤼~
Summary in Korean: 크리스마스 선물로 제가 busy에서 7일 지난 글 수정이 가능하게끔 고쳐두었습니다. 로컬 서버를 통해 테스트해보니 잘되네요. github에 pull request해놨고 review만 거치면 반영될거에요. 언제 될지는 모르겠지만. 최근 API변경때문에 한번 production code수정을 할거라 반영되기에 좋은 타이밍이 될지도. 반영되면 좋겠네요. 그래도 수많은 사람들이 쓰는 busy에 제가 도움이 되었다는 사실 자체만으로도 기쁠테니까요. 그럼 다들 메리크리스마스~!
대단히 감사합니다. 수고 하셨습니다.
메리 크리스마스~~
감사합니다. 크리스마스 잘 보내세요~~!
Merry Christmas.!!
정말 감사합니다. 누군가 만들어주기를 바랬는데. 고생하셨어요. 빨리 반영되었으면 좋겠어요.
메리크리스마스 에요~
감사합니다. 내친김에 다운보팅 slider랑 confirm도 추가하려했는데 된줄 알고 테스트했다가 그냥 라이크가 눌러졌네요ㅎㅎ 대충보니 이건 조금더 고칠게 많겠네요ㅎㅎ 전 모바일을 쓰지 않아서 사실 잘못 다운보팅을 누르지도 않긴한데 이거 두개가 현재 busy에 대한 가장 큰 불만인 것 같더라고요. 이건 안피곤님께 맡깁니다ㅎㅎ
블록체인스튜디오님은 이참에 busy.org의 컨트리뷰터로 활동하시는건가요? ㅎㅎ 저는 비지가 불편해서 사용을 잘 안했는데, 블록체인스튜디오님이 관심가지시니 저도 소스를 한번 봐야겠어요. ㅎㅎ
오 벌써 머지되고 서버에 올렸네요^^ 뿌듯뿌듯ㅎㅎ 워낙 간단한 수정이라ㅎㅎ 오 그럼 뭐 쓰시나요? 전 글쓸때는 모바일을 안써서. 아 맞다. steeve가 아마 안피곤님 원하시던 앱에 좀 가까울듯 AI기반으로 페북처럼 피드만들어주는 기능이 있대요. 초기단계라 버그들이 좀 있는 것 같지만.
저는 typora 앱에서 글을 작성합니다. 참고로 이미지는 스팀잇에 업로드해요. 그리고 디클릭을 통해 글을 등록한 다음, 마지막으로는 비지에서 한번 수정해요. 저는 항상 글을 이렇게 써요. ㅎㅎ 피드/보팅/댓글은
steemit.com을 사용합니다.그리고 steeve 는 오픈소스인가요? 구글 검색으로는 안나오네요.
아 맞다 dclick쓰려면 현재 글쓰는 것 자체는 디클릭에서 해야겠네요. 이미지는 디클릭 지원안되나요? 불안정하거나 혹시 추후 지원이 끊기거나 할때를 대비해서? 아니면 api로 업로드후 url쉽게 끌어다 쓰기 위함이려나요^^
steeve가 아직 오픈소스가 아닐거에요. steeve.app 아직 초기라서 steeve steemit 정도까진 쳐주셔야 검색이 그나마 되네요ㅎㅎ steempeak도 아쉽게 오픈소스가 아닌 것 같더라고요. busy의 가장 큰 공헌이 오픈소스라는 점일지도. 뒤에 나온 프로젝트들이 많이 참고했을 것 같은데.
맞아요. 이미지는 지원이 끊길 것을 대비해서 스팀잇에 올리고 있어요. 깃허브 위키 페이지에 올리는 방법도 가끔 사용합니다. 아티스팀은 이전에 업로드 했던 이미지가 모두 엑박으로 뜹니다. 다행이 스팀잇 이미지 서버에 모두 캐시되어 있어서 스팀잇 사이트에서는 잘보입니다.
그리고 steev.app 서비스 알려주셔서 감사합니다. 제가 원했던 기능이 있네요. 오픈소스가 아닌건 조금 아쉽네요.
크리스마스 잘 보내세요~
곰돌이가 @anpigon님의 소중한 댓글에 $0.018을 보팅해서 $0.005을 살려드리고 가요. 곰돌이가 지금까지 총 2231번 $28.256을 보팅해서 $27.740을 구했습니다. @gomdory 곰도뤼~
곰돌이가 @anpigon님의 소중한 댓글에 $0.017을 보팅해서 $0.006을 살려드리고 가요. 곰돌이가 지금까지 총 2216번 $28.075을 보팅해서 $27.576을 구했습니다. @gomdory 곰도뤼~
캬~! 완전 감사합니다~♥
행복한 목욜 ♥ 보내셔용~^^
Posted using Partiko Android
Hi @blockchainstudio!
Your post was upvoted by @steem-ua, new Steem dApp, using UserAuthority for algorithmic post curation!
Your UA account score is currently 4.257 which ranks you at #2704 across all Steem accounts.
Your rank has improved 61 places in the last three days (old rank 2765).
In our last Algorithmic Curation Round, consisting of 238 contributions, your post is ranked at #49.
Evaluation of your UA score:
Feel free to join our @steem-ua Discord server
Hey, @blockchainstudio!
Thanks for contributing on Utopian.
We’re already looking forward to your next contribution!
Get higher incentives and support Utopian.io!
Simply set @utopian.pay as a 5% (or higher) payout beneficiary on your contribution post (via SteemPlus or Steeditor).
Want to chat? Join us on Discord https://discord.gg/h52nFrV.
Vote for Utopian Witness!