Tim Abbott

Results 1350 comments of Tim Abbott

We should be sure to disable the widget when changing a user's avatar is disabled (see zulip/zulip#12251) because the organization is syncing them from LDAP.

In this case, it is unnecessary... you can just check for the keys in the register response, using false (not disabled) as the default if not present -Tim Abbott (mobile)...

@Ajnus the commit history needs to be cleaned up before this will be ready for review. Check out our [GitHub guide](https://zulip.readthedocs.io/en/latest/git-guide.html) and [commit guidelines](https://zulip.readthedocs.io/en/latest/contributing/commit-discipline.html) for more details.

Merged, thanks for all the work on this @sanchi-t!

We'll want to use https://zulip.com/api/get-stream-topics to get the full set of topics in the target stream. (In the web app, we show topics that the client knows about while waiting...

I got in a pretty complete review of this; I think there's still some things to figure out with how we want to handle the `rendered_markdown` display logic, which you...

I think that's a complete set of technical feedback for getting this PR into a mergeable state. Thanks for all the effort on this @Vector73!

I extracted and merged the backend of this as 9e4e85e1402378bf971b87c5f6223d7775db5fba, after the few small fixes noted above, which should make rebasing the remainder easy while we work through those UI...

I don't think deleting a lot of snippets at once is going to be very common, so the confirmation modal strategy is probably a good one. > I think this...