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

[FIX] Snippetted messages not working #8937

Merged
merged 5 commits into from
Dec 8, 2017
Merged

Conversation

karlprieb
Copy link
Contributor

@karlprieb karlprieb commented Nov 23, 2017

@RocketChat/core

Closes #8585 #8809

2017-11-23 18 06 24

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-8937 November 23, 2017 20:08 Inactive
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.

Name is not displayed anywhere, why still take it? Should probably be at least displayed along side message in panel.

@rodrigok rodrigok added this to the 0.60.0 milestone Dec 8, 2017
@rodrigok rodrigok temporarily deployed to rocket-chat-pr-8937 December 8, 2017 12:21 Inactive
@rodrigok
Copy link
Member

rodrigok commented Dec 8, 2017

@karlprieb @geekgonecrazy Lets merge this and add the title as a FIX while in release candidate? What do you think?

@rodrigok rodrigok temporarily deployed to rocket-chat-pr-8937 December 8, 2017 12:55 Inactive
@karlprieb
Copy link
Contributor Author

@rodrigok @geekgonecrazy I'm only fixed the snippetted messages and made it work as always (without title). I can implement that, but I agree with @rodrigok we can make on another PR.

@geekgonecrazy
Copy link
Contributor

follow up issue created #9050

@rodrigok rodrigok changed the title [FIX] snippetted messages not working [FIX] Snippetted messages not working Dec 8, 2017
@rodrigok rodrigok merged commit 4ba0170 into develop Dec 8, 2017
@rodrigok rodrigok deleted the snippetted-messages branch December 8, 2017 18:00
@ged-nn
Copy link

ged-nn commented Jun 21, 2018

please, you can use the full example of using snippets.
I just can not understand how it works.
:-(

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.

Can't add snippet in 0.59.1
5 participants