zulip-terminal icon indicating copy to clipboard operation
zulip-terminal copied to clipboard

Add notification checkboxes in Stream Info view.

Open zee-bit opened this issue 4 years ago • 4 comments

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:

  • [ ] Audible desktop notification
  • [ ] Email notification
  • [x] Visual desktop notification, and
  • [ ] Mobile notification

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 avatar Jan 23 '21 09:01 zee-bit

@zulipbot claim

zee-bit avatar Jan 23 '21 09:01 zee-bit

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!

zulipbot avatar Feb 16 '21 01:02 zulipbot

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

zee-bit avatar Jul 13 '21 20:07 zee-bit

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

zulipbot avatar Mar 22 '22 22:03 zulipbot