Ujjawal Modi
Ujjawal Modi
@alya please review this pr.
> Thanks! I found one behavior in manual testing that I think needs to be changed: > > 1. Unsubscribe a user from a stream > 2. Turn on the...
@alya I have started [CZO discussion](https://chat.zulip.org/#narrow/stream/101-design/topic/Resubscribing.20a.20stream.20which.20was.20unsubscribed) regarding the same.
@alya as discussed [CZO discussion](https://chat.zulip.org/#narrow/stream/101-design/topic/Resubscribing.20a.20stream.20which.20was.20unsubscribed) I have done the changes and updated the screenshot. Please have a look
@sahil839 please review this PR.
@sahil839 I have done the changes as per your comments. Please have a look.
@sahil839 I have done the changes as per your comments.
@sahil839 I have addressed your comments. Also now checkbox is disabled if `push_notifications` are not configured. The UI for disabled state is same as we have in personal settings. I...
@sahil839 bumping this PR for review, as it has been a while since last review.
> Posted a few comments. Also, I think you missed to reply or update the PR as per couple of previous comments. > > I also noticed a bug where...