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

Make posts/users sortable by chat activity #265

Closed
3 tasks
Tracked by #263
evan-scales opened this issue Feb 7, 2024 · 4 comments · Fixed by #437
Closed
3 tasks
Tracked by #263

Make posts/users sortable by chat activity #265

evan-scales opened this issue Feb 7, 2024 · 4 comments · Fixed by #437
Assignees
Labels
discuss Discuss something enhancement New feature or request med-priority
Milestone

Comments

@evan-scales
Copy link
Collaborator

evan-scales commented Feb 7, 2024

On the social page we should also allow the ability to sort users/posts based on activity of the chat. Chat already has a property for LastMessage - Message & LastMessageAt - DateTime. This filtering probably won't be too difficult for posts as each post has a chat, but it will be a bit more difficult for users.

For that I think we would need to add a nullable Chat property to UserRelation. Nullable because we don't need one when blocking a user or just sending a friend request. But we would still be able to message users you are not friends with. When a friend relation is made we either make a new chat for the relation, or find their existing chat. With that added we could now sort users based on chat activity

I think this stuff can wait untill after beta, if at all

AC

  • Make api able to sort posts/users based on their chat activity
  • Add sort selector to social, including sort options we currently use on discover + new chat activity option
  • Display the last message sent on user/post cards on social
@evan-scales evan-scales mentioned this issue Feb 7, 2024
7 tasks
@epadams epadams added the enhancement New feature or request label Feb 26, 2024
@jbytes1027 jbytes1027 added low-priority discuss Discuss something labels Mar 1, 2024
@evan-scales
Copy link
Collaborator Author

Hi @josemvidal I feel like we're lacking new features in the rc1 release. Is it okay to add this to rc1 release too?

@jbytes1027
Copy link
Contributor

IMO I think we should focus on more testing and commenting @evan-scales. If we get all our current and backlog issues done then maybe work on a new feature the week before rc1. Also, this would be a lot of work for a small usability feature.

@josemvidal
Copy link
Member

Yes, you can always add more if you want.

@evan-scales
Copy link
Collaborator Author

@jbytes1027 Our backlog is mostly filled with commenting (non-coding), testing (also kinda non-coding), and small quick fixes.

Considering this is our last chance to add new features, and almost all apps with a chat component automatically sort them by last activity, I will complete this for rc1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss Discuss something enhancement New feature or request med-priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants