Project Information
- Repository: https://github.com/jakipatryk/steeditor
- Project Name: Steeditor.app
- Publisher: @jakipatryk
A GitHub Issue was created:
The bug was confirmed by the project owner. Even though the issue is still open, the main problem was fixed by @jakipatryk, as can be seen by this post's metadata.
Expected behavior
steeditor should put URLs and user mentions from the post body in the corresponding json_metadata
fields
non-markdown links are not included in
json_metadata['links']
. This is a problem for the utopian templates, since they propose the target repository as non-markdown links, but the utopian-spreadsheet repo parsesjson_metadata['links']
for that string:
https://github.com/amosbastian/utopian-spreadsheet/blob/b4953e9af7910ffc3336a1945c3b31ae16aafb72/spreadsheet.py#L50
The target repository is not necessarily detected correctly in this case.user-mentions are found within markdown URLs. This is not necessarily a bug, only inconsistent to other interfaces.
How to reproduce
Two test posts with the same body: utopian template + one image + a markdown link
- from steeditor: steemd
{"app":"steeditor/0.1.0",
"format":"markdown",
"image":["https://steemitimages.com/0x0/https://user-images.githubusercontent.com/33018033/42889616-25f17de6-8aab-11e8-971e-e05abdb331e1.png"],
"tags":["not-utopian","not-analysis"],
"users":["stmdev"],
"links":["https://steemit.com/test/@stmdev/steeditor-test-1532283370486"],
"community":"steeditor"}
- from steemit steemd
{"tags":["not-utopian","not-analysis"],
"image":["https://steemitimages.com/0x0/https://user-images.githubusercontent.com/33018033/42889616-25f17de6-8aab-11e8-971e-e05abdb331e1.png"],
"links":["https://github.com/utopian-io/utopian.io","https://steemit.com/test/@stmdev/steeditor-test-1532283370486"],
"app":"steemit/0.1",
"format":"markdown"}
Differences:
- The steeditor metadata doesn't contain the repository in the
links
section, so the target repository may be empty or possibly wrong in the utopian review sheet. The steemit metadata contains the repository as the first entry inlinks
- The steeditor metadata detects a user mention in the markdown link for
@stmdev
, while steemit treats this as part of the URL and doesn't add ausers
field.
With the fix, the non-markdown links are properly detected now:
The repository is set as the first entry in the links
metadata and is, therefore, properly parsed by the utopian spreadsheet code.
Environment
steeditor.app as live on 2018-08-29
Chrome 68.0
Hi @stmdev thanks for making this contribution
I see the issue was fixed like you pointed out, in this commit made by you. This is a great find, it's the sort of detail that anyone can miss.
Feedback:
I look forward to your next contribution.
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]
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Thank you for your review, @fego!
So far this week you've reviewed 5 contributions. Keep up the good work!
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Hey, @stmdev!
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!
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Congratulations @stmdev! You have completed the following achievement on the Steem blockchain and have been rewarded with new badge(s) :
Award for the number of posts published
Click on the badge to view your Board of Honor.
If you no longer want to receive notifications, reply to this comment with the word
STOP
Do not miss the last post from @steemitboard:
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Hi @stmdev!
Your post was upvoted by @steem-ua, new Steem dApp, using UserAuthority for algorithmic post curation!
Your post is eligible for our upvote, thanks to our collaboration with @utopian-io!
Feel free to join our @steem-ua Discord server
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
oh @cryptoindex, wrong place for comment spam. Come on, you can do better than spamming the same 3 comments all over random Steem posts...
@steemflagrewards
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Steem Flag Rewards mention comment has been approved! Thank you for reporting this abuse, @crokkon.
Your comment has been repeated multiple times without regard to the post.
This post was submitted via our Discord Community channel. Check us out on the following link!
SFR Discord
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit