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

Bug: Incorrect Behaviour for delete Permission in Administration Settings #749

Open
SinghaAnirban005 opened this issue Dec 28, 2024 · 1 comment · May be fixed by #752
Open

Bug: Incorrect Behaviour for delete Permission in Administration Settings #749

SinghaAnirban005 opened this issue Dec 28, 2024 · 1 comment · May be fixed by #752
Labels
bug Something isn't working

Comments

@SinghaAnirban005
Copy link
Contributor

Description:

When setting delete permissions in Administration, the priority behavior for forcedeletemessage, deletemessage, and deleteOwnMessage is not functioning as expected
Specifically, users are able to view the delete icon if deleteOwnMessage is not set to true, even when the priority hierarchy is correctly configured

Steps to reproduce:

  1. Go to the Administration panel.
  2. Navigate to the Permissions section and configure delete permissions for a channel.
  3. Set forcedeletemessage to false for users.
  4. Set deletemessage to false for users
  5. Set deleteOwnMessage to false.
  6. Attempt to view the delete icon as a user in the channel.

Expected behavior:

When deleteOwnMessage is set to false, users should not see the delete icon for their messages.
When deleteOwnMessage is set to true, users should only see the delete icon for their own messages.
The permissions hierarchy should prioritize:

  1. forcedeletemessage
  2. deletemessage
  3. deleteOwnMessage

Actual behavior:

EmbeddedChat._.Simple.-.Simple.Storybook.-.Brave.2024-12-29.02-14-42.mp4
@SinghaAnirban005 SinghaAnirban005 added the bug Something isn't working label Dec 28, 2024
@SinghaAnirban005
Copy link
Contributor Author

Am working on it !! Will soon raise a PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant