-
Notifications
You must be signed in to change notification settings - Fork 79
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
Group Chat 1:n #34
Comments
Well, consider the following changes to models:
It would be a nice idea to create some indexes or auto-updated Materialized views with user's dialogs list . Then, adjust consumers code to handle the possibility of multiple users when receiving the message, add checks for InvalidGroupPk, and adjust the db saving accordingly. After that with things like administation you want to add "admins" field (subset of participants) to Group and add checks to send MessageTypes.TextMessage / FileMessage to check for banned / unbanned and various other permission-related stuff. Probably add a few new message updates for user joining a group / leaving a group, and add new message types to request group joining / leaving Well, probably mostly tedious work to add not only the possilibty of handling >2 user "dialogs" but also new features, such as administration, banning / restricting allowed communication types /joining or leaving the groups, probably forming public and private group links etc. |
Fantastic - many thanks for the quick answer! I looked into the changes and I would also need to change the main.js file (django_private_chat2-master\example\static\js\main.js) too. However this file is minified and therefore hard to work with. Is there any chance to get the unminified main.js? Thanks again! |
Hi, Minified main.js file is the result of |
I wanted to make an PR for chat frontend in repository/example for iOS app (Swift) |
Hello, |
Okay, cool. Will be shipping the code after completion |
Hi,
I would like to have a 1:n chat (group-chat) additionally to the current 1:1 chat. I was wondering what kind of modifications are needed to do so. Maybe you have same advice as for the starting point.
Many thanks!
The text was updated successfully, but these errors were encountered: