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

[NEW] Update message actions #14268

Merged
merged 11 commits into from
Apr 27, 2019
Merged

[NEW] Update message actions #14268

merged 11 commits into from
Apr 27, 2019

Conversation

MarcosSpessatto
Copy link
Contributor

@MarcosSpessatto MarcosSpessatto commented Apr 26, 2019

Closes #14254

Items to remove

  • Translate (Kept in message actions, only changed the label to "Translate" and "View Original")
  • Ignore User (Just removed, the Ignore actions were already in "User Info")
  • Snippet (Commented code for now, because the behavior of the snippets will be changed in the future. See: Send the message snippet as an attachment feature-requests#99)
  • Follow Thread (Just commented code)

@geekgonecrazy
Copy link
Member

Translate (Just commented code)

What will happen to this one? Are we going to remove this ability?

@MarcosSpessatto
Copy link
Contributor Author

MarcosSpessatto commented Apr 27, 2019

@geekgonecrazy Not exactly, we are moving the icon to another place only kept it in the same place, just changed the labels 🙂
#14254

@MarcosSpessatto MarcosSpessatto changed the title [WIP] [NEW] Update message actions [NEW] Update message actions Apr 27, 2019
@rodrigok rodrigok merged commit 64a9552 into develop Apr 27, 2019
@rodrigok rodrigok deleted the update-message-actions branch April 27, 2019 20:25
@rodrigok rodrigok mentioned this pull request Apr 28, 2019
@novacrazy
Copy link

Does reporting actually do anything but add it to the database? #1364 is still open, so I guess not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
5 participants