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] Reply/quote multiple messages #12171

Merged
merged 7 commits into from
Oct 2, 2018

Conversation

rssilva
Copy link
Contributor

@rssilva rssilva commented Sep 26, 2018

Fix #11943

It depends on #12095

reply-multiple

@rssilva rssilva added the ui/ux label Sep 26, 2018
@rssilva rssilva added this to the 0.71.0 milestone Sep 26, 2018
@rssilva rssilva self-assigned this Sep 26, 2018
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-12171 September 26, 2018 17:28 Inactive
@rssilva rssilva modified the milestones: 0.71.0, 0.70.0 Sep 26, 2018
@rssilva rssilva force-pushed the fix/attachment-on-reply branch from 483409b to 9c9f12c Compare September 26, 2018 17:49
Copy link
Contributor

@geekgonecrazy geekgonecrazy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume this is good from UX perspective?

@ggazzo ggazzo modified the milestones: 0.70.0, 0.71.0 Sep 27, 2018
@ggazzo ggazzo changed the title [FIX] Reply/quote multiple messages [NEW] Reply/quote multiple messages Oct 2, 2018
@CLAassistant
Copy link

CLAassistant commented Oct 2, 2018

CLA assistant check
All committers have signed the CLA.

@ggazzo ggazzo requested a deployment to rocket-chat-pr-12171 October 2, 2018 16:58 Abandoned
@ggazzo ggazzo requested a deployment to rocket-chat-pr-12171 October 2, 2018 17:05 Abandoned
@ggazzo ggazzo merged commit 7dc2fca into fix/attachment-on-reply Oct 2, 2018
@ggazzo ggazzo deleted the fix/reply-multiple-messages branch October 2, 2018 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants