Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix-reblogged-by #2150

Merged
merged 3 commits into from
Jan 18, 2019
Merged

fix-reblogged-by #2150

merged 3 commits into from
Jan 18, 2019

Conversation

economicstudio
Copy link
Collaborator

@economicstudio economicstudio commented Jan 13, 2019

Fixes #2149

Changes

  • Fix reblogged_by

post.first_reblogged_by returns null. post.reblogged_by should be used as follows:
https://github.com/steemit/condenser/blob/7430aca58ae9f86e501f52e3cf8dba5efc9fe3e0/src/app/components/cards/PostSummary.jsx#L70

Test plan

Explain how to test changes you made.

  • Go to feed
  • Compare it with the feed on Steemit.com

Demo

Busy.org: my feed (rosatravels isn't my following)

Steemit.com shows the reblogged information correctly

After the fix on my local dev server

In addition, to show 'reblogged' in blog (not feed), the current code should be changed. Currently there is no easy way to differentiate if it's feed or blog. But any resteemed post in a blog is easily noticeable by author's name, so not important as reblogged_by in feed.

post.first_reblogged_by returns null.

should use post.reblogged_by

https://github.com/steemit/condenser/blob/7430aca58ae9f86e501f52e3cf8dba5efc9fe3e0/src/app/components/cards/PostSummary.jsx#L70

In addition, to show 'reblogged' in blog (not feed), the current code should be changed. Currently there is no easy way to differentiate if it's feed or blog. But any resteemed post in a blog is easily noticeable by author's name, so not important as reblogged_by in feed.
@Sekhmet Sekhmet temporarily deployed to busy-develop-pr-2150 January 17, 2019 15:35 Inactive
Copy link
Contributor

@Sekhmet Sekhmet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@Sekhmet
Copy link
Contributor

Sekhmet commented Jan 17, 2019

@economicstudio
You have to check this checkbox on the right to let us edit this branch:
image

@economicstudio
Copy link
Collaborator Author

economicstudio commented Jan 18, 2019

Thanks for your approval and I made it (and all others) editable.

@Sekhmet Sekhmet merged commit 119cd24 into busyorg:develop Jan 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

reblogged_by is not shown
2 participants