zulip / zulip-terminal

Official Zulip terminal client.
Apache License 2.0
666 stars 253 forks source link

Add notification checkboxes in Stream Info view. #887

Open zee-bit opened 3 years ago

zee-bit commented 3 years ago

This issue extends PR #880 and should work towards improving the Stream Info view in ZT. Currently, there are just two checkboxes in StreamInfoView(mute_stream & pin_to_top). These are fine but we want to add more checkboxes related to Stream notifications. We could potentially add:

The logic could be borrowed from zulip/zulip or those other checkboxes already present in ZT. Zulip's API documentation should prove helpful in tackling this issue.

@neiljp I have opened this issue(to keep track of the tasks that needs to be done, and) as a continuation of the work I already did in #880

NOTE These require underlying fetching handling updates of the settings, in addition to the UI, ie. we expect to synchronize these with the server (though we might have our settings in future) -- neiljp

zee-bit commented 3 years ago

@zulipbot claim

zulipbot commented 3 years ago

Hello @zee-bit, you claimed this issue to work on it, but this issue and any referenced pull requests haven't been updated for 10 days. Are you still working on this issue?

If so, please update this issue by leaving a comment on this issue to let me know that you're still working on it. Otherwise, I'll automatically remove you from this issue in 4 days.

If you've decided to work on something else, simply comment @zulipbot abandon so that someone else can claim it and continue from where you left off.

Thank you for your valuable contributions to Zulip!

zee-bit commented 3 years ago

Sorry for the "Fixes" comment on the PR, this isn't completely resolved yet. Re-opening!

zulipbot commented 2 years ago

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