-
-
Notifications
You must be signed in to change notification settings - Fork 657
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
Align message content deletion experience with web app #5528
Comments
An implementation note for this part:
What's happening here is that for editing a message we re-use the UI (in (The server already accepts an empty message content when editing a message; it then transforms that internally to say "(deleted)" instead. In fact editing a message to be empty is exactly what we do in the existing "Delete message" option, the one we'll be removing in the first part of this issue.) |
Hi @alya, I was wondering if you are still looking for help to solve this issue. I looked into it, and I think I might have an idea on how to fix it. I made some changes to my forked repository, that allows users to delete message when deleting the content of a message when editing messages. Basically, I use the isEditing prop in the ComposeBox component to keep track of when the message is being edited or deleted. If its being edited, I ignore the "message is empty" error. That allows the user to send empty content while editing the message. I also changed the sendCallback in the ChatScreen to allow empty content to be sent through the API. I just had to change the if condition. Im still working on removing the delete message button. Could you assign this issue to me? Please let me know if you need anything else. Regards, |
I also have an idea on how to remove the delete button for users who are not admins. I just need to check if the user is not an admin, and if it is an admin, I will add the delete button to the options of the action sheet, else I wont add it. I understand how to remove it for users, but I also see it being added when message.isOutbox is True. Im not really sure what isOutbox means. I would appreciate it if I could get some help. Regards, |
@gnprice @chrisbobbe Any feedback on the approach described above? |
@KshitizSareen If you have questions as you are working on an issue, you'll generally get a faster response if you post them in the #mobile-team stream in the Zulip development community. Be sure to follow https://zulip.readthedocs.io/en/latest/contributing/asking-great-questions.html. Thanks! |
Hi @alya, Thank you so much. |
We should make two changes to make the mobile app message deletion experience consistent with the web app and https://zulip.com/help/edit-or-delete-a-message#delete-a-message. Specifically, we should only use "Delete message" to refer to fully removing the message, rather than deleting message content.
We should:
The text was updated successfully, but these errors were encountered: