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

Allow hiding moderation actions in streamer mode #3926

Merged
merged 8 commits into from
Nov 6, 2022

Conversation

Mm2PL
Copy link
Collaborator

@Mm2PL Mm2PL commented Aug 15, 2022

Pull request checklist:

  • CHANGELOG.md was updated, if applicable

Description

This adds a new setting to streamer mode which will hide any moderation messages. This doesn't hide moderated messages, that could be another PR. This hides any messages which have the Timeout or Untimeout flags, exactly like the Show moderation messages setting. Also of note: this won't hide timeout responses for commands you run.
Closes #2729.

@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@Mm2PL Mm2PL requested a review from zneix August 15, 2022 23:13
zneix
zneix previously requested changes Aug 15, 2022
Copy link
Collaborator

@zneix zneix left a comment

Choose a reason for hiding this comment

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

I'm surprised someone actually bothered requesting my - review on anything... few last merges definitely show like you don't need me here

people just approve changes like this here without even paying attention to details, while also disregarding me or my opinion, then complain about quirks that exist in master because "somehow that mistake got merged in.." 🙄

src/singletons/Settings.hpp Outdated Show resolved Hide resolved
src/messages/layouts/MessageLayout.cpp Outdated Show resolved Hide resolved
@jupjohn
Copy link
Contributor

jupjohn commented Aug 16, 2022

I'm surprised someone actually bothered requesting my - review on anything... few last merges definitely show like you don't need me here

people just approve changes like this here without even paying attention to details, while also disregarding me or my opinion, then complain about quirks that exist in master because "somehow that mistake got merged in.." 🙄

@zneix This comment has nothing to do with the review of this code. Mistakes happen, we fix them and move on - we're human. It's also inconsiderate to the other maintainers/developers of chatterino to claim they don't pay attention to the details when reviewing additions to the project. I'm sure they do in fact care about the quality of what's going in to the codebase as does the community because they care about the software they use.

You chose to step away from the project for a bit which is understandable and respectable - but to come back and claim that your opinion isn't respected by the community or that they're actively disregarding your input is ignorant of the fact that you made that choice to back off.

I personally find your skills and knowledge of chatterino's domain valuable. The drama however, is annoying and doesn't contribute anything positive to the developer community. I would encourage you to think about how you're impacting the community by exhibiting this sort of behaviour and grow from it.

@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2022

clang-tidy review says "All clean, LGTM! 👍"

@pajlada pajlada enabled auto-merge (squash) November 6, 2022 13:29
@pajlada pajlada disabled auto-merge November 6, 2022 13:29
@pajlada pajlada requested a review from zneix November 6, 2022 13:30
@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2022

clang-tidy review says "All clean, LGTM! 👍"

src/singletons/Settings.hpp Outdated Show resolved Hide resolved
@Mm2PL Mm2PL enabled auto-merge (squash) November 6, 2022 16:09
@Mm2PL Mm2PL merged commit df4c294 into master Nov 6, 2022
@Mm2PL Mm2PL deleted the feature/hide_mod_actions_in_streamer_mode branch November 6, 2022 16:30
@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2022

clang-tidy review says "All clean, LGTM! 👍"

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.

Show moderation messages part of streamer settings
5 participants