Skip to content

Commit

Permalink
views: Remove muted_users from user_list.
Browse files Browse the repository at this point in the history
Skip user from showing in user list if user is muted.
Test updated.

Co-authored by: Subhasish-Behera <greendoor3000@gmail.com>
  • Loading branch information
rsashank committed Sep 20, 2024
1 parent 3937ee2 commit ed83f2b
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions tests/ui/test_ui_tools.py
Original file line number Diff line number Diff line change
Expand Up @@ -1078,6 +1078,7 @@ def test_users_view(self, users, users_btn_len, editor_mode, status, mocker):
user_btn = mocker.patch(VIEWS + ".UserButton")
users_view = mocker.patch(VIEWS + ".UsersView")
right_col_view = RightColumnView(self.view)
self.view.model.is_muted_user.return_value = False
if status != "inactive":
user_btn.assert_called_once_with(
user=self.view.users[0],
Expand Down
4 changes: 4 additions & 0 deletions zulipterminal/ui_tools/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -726,6 +726,10 @@ def users_view(self, users: Any = None) -> Any:

users_btn_list = list()
for user in users:
with open("tx.txt", "a") as f:
f.write(str(self.view.model.is_muted_user(user["user_id"])))
if self.view.model.is_muted_user(user["user_id"]):
continue
status = user["status"]
# Only include `inactive` users in search result.
if status == "inactive" and not self.view.controller.is_in_editor_mode():
Expand Down

0 comments on commit ed83f2b

Please sign in to comment.