Alya Abbott

Results 1722 comments of Alya Abbott

Automatically detect the user’s time zone

The profile settings button on the confirmation toast seems unnecessary. Let's drop it. The confirmation is also missing a `.` at the end.

channel: Add option to notify users newly added to a channel.

@pratik-pc does this look good to you now?

channel: Add option to notify users newly added to a channel.

Yes, thanks for flagging! @sahil839 perhaps you could review this one?

channel: Add option to notify users newly added to a channel.

Thanks! Looks good overall, but let's tweak a few things: 1. Let's just put the setting on the line below the heading. I think that will look best in general,...

channel: Add option to notify users newly added to a channel.

Looks good to me in manual testing otherwise! You can tag for integration review once the comment above has been addressed.

channel: Add option to notify users newly added to a channel.

What happens if a limited guest user gets added to a channel by someone whose name they don't have access to? (I haven't checked if this was already discussed above.)...

channel: Add option to notify users newly added to a channel.

Thanks! The screenshots in the PR description seem to be out of date now?

channel: Add option to notify users newly added to a channel.

I guess the narrow one is still stale?

notification: Use existing email format for missed 1:1 DM via DM group.

@mateuszmandera Are you up for reviewing this one?

Issue 34519 allow admin to modify user profile picture

Thanks! Please clean up your commit history and post again to request a review. See [here](https://zulip.readthedocs.io/en/latest/contributing/commit-discipline.html) for guidelines. You will also need to include still screenshots of all your changes,...