-
-
Notifications
You must be signed in to change notification settings - Fork 252
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
Add guest user suffix, depending on organization setting #1444
Comments
@neiljp can i pick this up ? |
@AnkurPrabhu Sure! Apologies, I didn't see your message earlier! |
This is implemented in mobile via zulip/zulip-mobile#5804, and based on the release announcement on chat.zulip.org, hides the guest suffix only when on 8.0+ and the organization has the setting to do so. There are also screenshots that suggest places that will need updating in zulip/zulip-mobile#5809. |
@zulipbot claim |
Welcome to Zulip, @AnkurPrabhu! We just sent you an invite to collaborate on this repository at https://github.com/zulip/zulip-terminal/invitations. Please accept this invite in order to claim this issue and begin a fun, rewarding experience contributing to Zulip! Here's some tips to get you off to a good start:
As you work on this issue, you'll also want to refer to the Zulip code contribution guide, as well as the rest of the developer documentation on that site. See you on the other side (that is, the pull request side)! |
Hey @neiljp just wanted to say that i am still working on this issue some of the changes are done and i will raise a pr and if you approve it then i will start with the testing part |
@neiljp please take a look at the pr, The tests are failing as I have not fixed them because i wanted to know if my approach is right and also wanted to know other places where the change needs to be done and have missed it . will fix the test with the review changes |
@AnkurPrabhu You have been unassigned from this issue because you have not made any updates for over 14 days. Please feel free to reclaim the issue if you decide to pick up again. Thanks! |
From feature level 216 (Zulip 8.0),
realm_enable_guest_user_indicator
is present in the response fromregister
(and in events), which should control whether we add a(guest)
suffix to users with that role.The web app UI needs checking for which locations this appears, which likely needs a development environment in order to explore behavior for when this setting is enabled or otherwise.
We should also confirm that
However, this will also need tests, which may need to be feature level (ZFL) dependent.
The text was updated successfully, but these errors were encountered: