Skip to content
This repository has been archived by the owner on Aug 11, 2024. It is now read-only.

Implement 'likes' #1106

Merged
merged 19 commits into from
Feb 14, 2024
Merged

Implement 'likes' #1106

merged 19 commits into from
Feb 14, 2024

Conversation

bfollington
Copy link
Collaborator

@bfollington bfollington commented Feb 7, 2024

  • Likes are stored in _likes_.json
  • Managed by UserLikesService
  • Likes are listed on your profile publicly in a new "Likes" tab
  • Notes can be liked directly from the Deck view
  • Notes can be liked from the meta sheet in the editor/viewer
    • OR, alternatively, from the button in the bottom right mirroring the Deck view cards

There is obvious room for improvement by incorporating likes into the DB index (#1111) and, as an extension, building content discovery (via likes-of-friends). There may also be performance issues associated with loading previews for a large number of likes.

We have also discussed changing the way we handle hidden files (#1112), which affects this feature.

Questions

  • Do we want to display like status in list views?
  • Should you be able to like notes from list views?

Demo

Screen.Recording.2024-02-12.at.4.53.23.pm.mov

Copy link
Collaborator

@gordonbrander gordonbrander left a comment

Choose a reason for hiding this comment

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

LGTM 🥳

This is amazing!

@bfollington bfollington merged commit 25f4459 into main Feb 14, 2024
1 check passed
bfollington added a commit that referenced this pull request Feb 14, 2024
Follow on from
#1106

Introduces:
- Show liked status with a small heart in list views (notebook &
profile)
- Add / remove likes from the list view context menu
- A new `EntryNotification` abstraction to encode the core set of
actions that can be taken on a note

# Demo


https://github.com/subconsciousnetwork/subconscious/assets/5009316/b4cc4bda-049f-4f62-9dcb-ae4ab8858be2
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants