-
-
Notifications
You must be signed in to change notification settings - Fork 429
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
Show sidebar on navigating to Settings page #1084
base: main
Are you sure you want to change the base?
Conversation
@andersk could you please review this? |
16bb7a4
to
468676b
Compare
Fixes zulip#1077 Signed-off-by: tarun8718 <tarunkumar8718@gmail.com>
Hey @tarun8718 Thanks for this PR. One question I have in mind is why are we adding this piece of code for accounts with only one organization, where if you have multiple, you'll probably still face the same issue? |
@tarun8718 Let me know if you need a hand or if we should continue the work as you've already done the most. |
Signed-off-by: tarun8718 tarunkumar8718@gmail.com
What's this PR do?
Shows Sidebar(if disabled) on navigating to the settings page(for users with single-organizations).
Fixes: #1077
You have tested this PR on: