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

Add guest user suffix, depending on organization setting #1444

Open
neiljp opened this issue Nov 29, 2023 · 8 comments
Open

Add guest user suffix, depending on organization setting #1444

neiljp opened this issue Nov 29, 2023 · 8 comments
Labels
area: UI General user interface update help wanted This issue should be clear enough to work on version parity: 8

Comments

@neiljp
Copy link
Collaborator

neiljp commented Nov 29, 2023

From feature level 216 (Zulip 8.0), realm_enable_guest_user_indicator is present in the response from register (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

  • the suffix wasn't present unconditionally prior to this
  • this behaves correctly in a backwards-compatible way

However, this will also need tests, which may need to be feature level (ZFL) dependent.

@neiljp neiljp added help wanted This issue should be clear enough to work on area: UI General user interface update version parity: 8 labels Nov 29, 2023
@AnkurPrabhu
Copy link
Collaborator

@neiljp can i pick this up ?

@neiljp
Copy link
Collaborator Author

neiljp commented Jan 6, 2024

@AnkurPrabhu Sure! Apologies, I didn't see your message earlier!

@neiljp
Copy link
Collaborator Author

neiljp commented Jan 6, 2024

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.

@AnkurPrabhu
Copy link
Collaborator

@zulipbot claim

@zulipbot
Copy link
Member

zulipbot commented Jan 6, 2024

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)!

@AnkurPrabhu
Copy link
Collaborator

AnkurPrabhu commented Jan 9, 2024

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

@AnkurPrabhu
Copy link
Collaborator

AnkurPrabhu commented Jan 12, 2024

@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

@zulipbot
Copy link
Member

zulipbot commented Mar 15, 2024

@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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: UI General user interface update help wanted This issue should be clear enough to work on version parity: 8
Projects
None yet
Development

No branches or pull requests

3 participants