UX Problem: CHAT INVITE

thruthesky / fireflutter

Flutter Fire Second Version

MIT License

11 stars 6 forks source link

UX Problem: CHAT INVITE #161

Open withcenter-dev2 opened 3 months ago

withcenter-dev2 commented 3 months ago

Questions:

If password exist, should invite button be hidden?


Problem:
If password exist, based on the rules, it will not be allowed to write in chat rooms, unless it is leaving (or removing the current users uid).
I am not sure if invite should be disabled or what.

withcenter-dev2 commented 3 months ago

Before:

After:

withcenter-dev2 commented 3 months ago

Removed the newData.hasChild(auth.uid) === false

Once the auth.uid in users, they can add uids in the users (this is how we invite/join in chat-room)

However, upon critical checking, this rule will allow anyone (who is in chat-room users), to remove other members as well. The only uid that can be removed should be auth.uid.

This is for review.