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

Improving rolls with meme replies #19

Open
wants to merge 46 commits into
base: master
Choose a base branch
from

Conversation

Roujo
Copy link
Contributor

@Roujo Roujo commented Feb 19, 2021

No description provided.

gravufo and others added 30 commits March 19, 2018 22:21
…llib3-1.24.2

Bump urllib3 from 1.22 to 1.24.2
…quests-2.20.0

Bump requests from 2.18.4 to 2.20.0
…rkzeug-0.15.3

Bump werkzeug from 0.14.1 to 0.15.3
…equests-2.22.0

Bump requests from 2.20.0 to 2.22.0
…erkzeug-0.16.0

Bump werkzeug from 0.15.3 to 0.16.0
@gravufo
Copy link
Member

gravufo commented Mar 22, 2021

This will result in very heavy rolling sessions. I don't think I can approve this PR in its current form.

@Roujo
Copy link
Contributor Author

Roujo commented Mar 22, 2021

Yeah, it would be rather heavy wouldn't it. I'll rework it so that only special numbers are output as memes.

As an aside, there is work being done to make images created by the meme plugin collapsible, which would also help address this.
mattermost-community/mattermost-plugin-memes#40

All that would be left is the fact that they don't appear on mobile, which is still a blocking issue that I'll try to address somehow. ^^

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.

4 participants