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.

Sign up for GitHub

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

Jump to bottom

The 'Report Message' feature should hide on user's own message and enhance Report-Message Modal. #904

Open
thepiyush-303 opened this issue Jan 15, 2025 · 6 comments · May be fixed by #907
Open

The 'Report Message' feature should hide on user's own message and enhance Report-Message Modal. #904

thepiyush-303 opened this issue Jan 15, 2025 · 6 comments · May be fixed by #907
Labels
bug Something isn't working

Comments

@thepiyush-303
Copy link
Contributor

thepiyush-303 commented Jan 15, 2025 •
edited
Loading

Description:

The 'Report Message' feature should hide on user's own message and enhance Report-Message Modal.

Steps to reproduce:

  1. check on message toolbar
  2. you can report you own messages.

Expected behavior:

  • You should not be able to report own messages
  • message description should be included in report message modal.
  • include some styling in the modal.

Actual behavior:

Screencast.from.2025-01-15.22-14-36.webm

working on it.
@thepiyush-303 thepiyush-303 added the bug Something isn't working label Jan 15, 2025
@thepiyush-303 thepiyush-303 changed the title The 'Report Message' feature should hide the user's own message and enhance Report-Message Modal. The 'Report Message' feature should hide on user's own message and enhance Report-Message Modal. Jan 15, 2025
@MohitGoyal09
Copy link

Pls assign me this issue i can work on it

@thepiyush-303
Copy link
Contributor Author

Currently, I am working on it.
Once my PR get reviewed and not get selected, then you can make your changes for it.

@MohitGoyal09
Copy link

Okay

@dhairyashiil
Copy link
Contributor

Pls assign me this issue i can work on it

Hello @MohitGoyal09 ,
No one assigns issues here. Yes, there are other organizations where issues are assigned to people.

However, if you see an open issue with no PR attached to it, feel free to start working on it without asking.
I can see that no one has attached a PR to this issue yet, so you can go ahead and work on it.
But if an issue already has a PR, please don't work on it, as there is a high possibility that it will get merged

@MohitGoyal09
Copy link

Pls assign me this issue i can work on it

Hello @MohitGoyal09 ,
No one assigns issues here. Yes, there are other organizations where issues are assigned to people.

However, if you see an open issue with no PR attached to it, feel free to start working on it without asking.
I can see that no one has attached a PR to this issue yet, so you can go ahead and work on it.
But if an issue already has a PR, please don't work on it, as there is a high possibility that it will get merged

Okay. Will keep that in mind

@dhairyashiil dhairyashiil mentioned this issue Jan 15, 2025
Closed
3 tasks
@SinghaAnirban005
Copy link
Contributor

hey @dhairyashiil ,
While it's great to see your interest, I recommend checking if someone is already actively working on the issue
If an issue already has someone working on it, it’s better to coordinate with them to avoid duplicated efforts as piyush has mentioned that he is working on it

@thepiyush-303 thepiyush-303 linked a pull request Jan 15, 2025 that will close this issue
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
4 participants
@dhairyashiil @MohitGoyal09 @SinghaAnirban005 @thepiyush-303