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

settings: Fix action column in users table. #29986

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

Conversation

dhruv-goyal-10
Copy link
Collaborator

Fixed #29894

Fixes: Action column in users & deactivated users table.

@zulipbot
Copy link
Member

zulipbot commented May 7, 2024

Hello @zulip/server-settings members, this pull request was labeled with the "area: settings UI" label, so you may want to check it out!

@dhruv-goyal-10
Copy link
Collaborator Author

You can have a look at the updated UI here.

Screen.Recording.2024-05-07.at.8.06.37.PM.mov

@alya
Copy link
Contributor

alya commented May 7, 2024

Thanks for the fix! Please complete the self-review checklist (as applicable), and post a comment to request a review when ready.

@alya
Copy link
Contributor

alya commented May 7, 2024

How is the issue you noted in #29894 (comment) handled?

@dhruv-goyal-10
Copy link
Collaborator Author

How is the issue you noted in #29894 (comment) handled?

So, Initially whether to show last_active_date in row or not depended on the attribute is_active.

But, now I created an explicit attribute named as display_last_active_in_table which I am setting to true in case of Users Table and setting it to false in case of Deactivated Users Table.

So, now last_active_date will be visible in Users Table always and will never be visible in Deactivated Users Table irrespective of whether user is active or not.

@dhruv-goyal-10
Copy link
Collaborator Author

@alya Can you have a look at this PR?

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix action column in deactivated users table.
3 participants