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

Implemented delete button next to edit and modal confirmation #228

Merged
merged 4 commits into from
Jun 26, 2015

Conversation

rockhouse
Copy link
Contributor

Once more as discussed with @sampaiodiego now adjusted to new message rendering
Link to old PR #189

@sampaiodiego
Copy link
Member

thanks again @rockhouse =D

can you please add translations to confirm messages? the possibility for translation I mean.

@sampaiodiego
Copy link
Member

I've tested this and I got two things:

  • The message is not deleted real time to everyone. If you open two browsers and you delete a message, it will not be deleted in the another browser. It only disappear if you refresh the window.
  • I think you should not be able do delete system generated messages, eg. "user has joined the channel"

@rockhouse
Copy link
Contributor Author

Both suggestions by @sampaiodiego have been implemented in SHA: f8e542e

@sampaiodiego sampaiodiego merged commit 52ec9fe into RocketChat:master Jun 26, 2015
tkurz pushed a commit to redlink-gmbh/Rocket.Chat that referenced this pull request Mar 13, 2018
tkurz pushed a commit to redlink-gmbh/Rocket.Chat that referenced this pull request Mar 13, 2018
tkurz pushed a commit to redlink-gmbh/Rocket.Chat that referenced this pull request Mar 13, 2018
…-run-tests-separately

Run tests separately
HappyTobi pushed a commit to HappyTobi/Rocket.Chat that referenced this pull request Jul 10, 2018
set "available" status from system idle time
shubhsherl pushed a commit to shubhsherl/Rocket.Chat that referenced this pull request Jul 21, 2020
Add pushm and pushm_encrypted to Message object
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants