Tim Abbott

Can you clean up the commit history for this pull request? The git guide has step-by-step directions for how to do this, and Google can help you with how to...

I think we need a bit more margin between the heading and the status code. I also am not sure about the removal of the heading text for the common...

This is great! @vinitS101 @showell would you be up for doing a review?

For automated testing, you can use `mock.patch` to wrap `fail_with_message` to first call `do_deactive_user` and then call the original function in the scope of a single test. (For manual testing,...

This is great, but the test for whether the previous message was one of these "marked topic as resolved" notifications is pretty hacky. I think we want to address that...

I added a minimal set of commits to create a `message_type` field and use that for this feature. I don't think the last two commits are ready to merge; the...

Yeah, please do!

@riteshsoni123 thanks for working on this! Can you squash the commits and write a proper commit message that explains the reasoning behind these changes? Check out the Zulip commit message...

@Jagansivam28 would you be up for helping review this?

@alya can you review this?