Niloth P
Niloth P
Nice work! Thanks for addressing those. The motivation and code LGTM. As discussed, this is not something that I can run and test though, as I don't use WSL.
@Aditya8840 Did you forget to push your changes after Satyam Bansal's review? The comments have been marked as resolved without any updates.
This looks good to me, thanks for all the work, @Aditya8840 . Thanks for the reviews, Pieter and Satyam Bansal. - It looks like after splitting this into several commits,...
I've made the following changes: #### 1st commit - [x] Re-ordered to `s`, `S`, `z`, `Meta .` - [x] User-facing category name: "Switch message view" -> "Switching messages view" -...
Note: The "Narrow to compose box message recipient" command that we just assigned to category "compose_box" in #1520 is now re-assigned to category "narrowing". Comparing the 2 categories, I think...
1. Grouped 'p' and 'n' into the 'narrowing' category. 2. Oops, thank you! I've rephrased it to "Switch message view to the compose box target". I felt 'recipient' could be...
Hi, @axllbk! Any updates? Feel free to put up a draft PR if you're almost there but not quite :)
@TusharMohapatra07 I do have a couple of local WIP branches, but I haven't looked at it in a few weeks. I think it was mostly functional, but I probably need...
@Eeshu-Yadav Did you miss [this message](https://github.com/zulip/python-zulip-api/issues/831#issuecomment-2532216889) in this thread? I don't have the permission to re-assign folks. You can just proceed with your contribution and create a PR, without waiting...
> P for DM feed functionality is currently removed, even from message context; it would be useful to retain this feature generally? [discuss] I don't see why we'd need to...