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

Repost feature #2

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Zauzolkov
Copy link

@Zauzolkov Zauzolkov commented Nov 20, 2022

Basic info

This pull-request offers repost feature (#1).

Reposts are shown together with regular posts in consistent timeline.

Details

Proposed terminology:

  • Repost – means only event of repost, without body of original post (there is only link to origin).
  • Origin – original post, which can be retrieved by link from repost.

Proposed Changes

DB Schema changes: Add repost table with columns post_id, date_added and repost_uri (link to origin).

When timeline is loading, body and other data of reposts are populated from the origin, with addition of some metadata from repost itself.

From the user's point of view, "repost" button behaves almost like a "like" button, but it also adds post to timeline of user.

TODO

  • Detailed testing is required
  • Use CSS classes instead of hard-coding
  • Better styling of "reposted by" badge?
  • Compatibility with older clients?
  • Bug: Like on repost is probably broken (it "likes" repost itself instead of origin post)
  • Ability to hide reposts? Squash multiple reposts (by many users) into one?

Demo

screen-2022-11-20_12.40.44.mp4

⚠️ WARNING ⚠️

This is a draft which is not properly tested. Don't use this modified client with your main profile! You should create a new profile for tests (or make backups) to avoid possible data loss.

ZeroNet demo page (expect seeding issues): http://localhost:43110/1AXjiwipCneqUXThCYHKnrCijyMsFBCN2o/

@Zauzolkov Zauzolkov mentioned this pull request Nov 20, 2022
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.

1 participant