issue: https://linear.app/comm/issue/ENG-3176/make-pressing-a-drawer-item-in-chat-tab-filter-available-chats
Using in CommunityDrawerItem and CommunityDrawerItemCommunity handler implemented im previous diffs. This allows to entierly remove CommunityDrawerItemChat component, as it's only function
is now supplied by the handlers.
Details
Details
Run web app, checked that pressing drawer items results in:
- navigating to the corresponding chat, changing the list of availabe chats in ChatThreadList, and expanding the community, for community items in the Chat tab
- navigating to the corresponging chat, for non-community items in the Chat tab
- changing the list of available filters in the Calendar tab
some corner cases:
- If the user is in a subchannel, and presses on the community, we only navigate to that community chat, since I feel it would be weird for that community to collapse in this case
- If user is in the community chat and presses on that community, the community collapses, and the list of chats gets unfiltered (the filter is cleared)
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
web/sidebar/community-drawer-item-community.react.js | ||
---|---|---|
27–28 ↗ | (On Diff #24791) | I assume this doesn't work for some reason? |
web/sidebar/community-drawer-item-community.react.js | ||
---|---|---|
28 ↗ | (On Diff #26768) | The point of my suggestion to remove the event parameter was so you could delete this eslint-disable line. Is that not possible for some reason? |
web/sidebar/community-drawer-item-community.react.js | ||
---|---|---|
28 ↗ | (On Diff #26768) | Ohh, right, sorry. This is not needed |
web/sidebar/community-drawer-item.react.js | ||
---|---|---|
36–37 ↗ | (On Diff #26808) | Same thing here? |