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

Feat/reblog stats #1403

Open
wants to merge 8 commits into
base: development
Choose a base branch
from
Open

Feat/reblog stats #1403

wants to merge 8 commits into from

Conversation

Adesojisouljay
Copy link
Collaborator

@Adesojisouljay Adesojisouljay commented Jul 15, 2023

What does this PR?

In this PR, I added reblog stats that shows a list of users who reblogged a post.
Users can see number of reblogs a post has and a list icon that when clicked shows a list of users who reblogged the particular post.

Github issue

reblogStats.mp4

Video/ Screenshot

@feruzm
Copy link
Member

feruzm commented Jul 17, 2023

@Adesojisouljay PR description and video missing.

@Adesojisouljay
Copy link
Collaborator Author

@Adesojisouljay PR description and video missing.

Need to add couple changes, i'll add video and description
Thanks

@feruzm
Copy link
Member

feruzm commented Jul 20, 2023

@Adesojisouljay you can add number next to existing reblog button. No need extra icon, only when logged in user reblogs, icon changes the color, that's it. Otherwise, we can always show number of reblogs and reblogged people on Post page.

@Adesojisouljay
Copy link
Collaborator Author

Okay, noted, we should just show number of reblogs. What about those lists, should we show them when number is clicked or we look for some on page to list those who reblogged a post?

@feruzm
Copy link
Member

feruzm commented Jul 20, 2023

@Adesojisouljay Yes clicking on number should show list, clicking on icon should (un)reblog

@Adesojisouljay
Copy link
Collaborator Author

@Adesojisouljay Yes clicking on number should show list, clicking on icon should (un)reblog

Alright, that's noted.
Thanks

@feruzm feruzm added the onhold label Jul 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants