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 how chat recent actions filter selection logic works #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pegurov
Copy link

@pegurov pegurov commented Jun 23, 2019

There is a bug in the channel system messages filter.
Channel -> Administrators -> Recent actions -> All actions (screen title)
If you deselect one of the action types, and then tap on the 'All actions' switch, all actions become deselected. 'Done' button becomes inactive, preventing you to apply the filter. I made a fix for this bug.

There is another bug in the channel system messages filter.
Channel -> Administrators -> Recent actions -> All actions (screen title)
If you just open the screen, and try to deselect one of the admins in the 'ADMINS' section, the FIRST TAP will do nothing. You can then tap on the 'All admins' switch above, and it also will do nothing.
I fixed the issue, so that the admins filter always works as expected.

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.

1 participant