Skip to content

[NEW] Add Discussions to RoomActionsView#3287

Merged
gerzonc merged 92 commits intodevelopfrom
new.add-discusions-roomactionsview
Feb 1, 2022
Merged

[NEW] Add Discussions to RoomActionsView#3287
gerzonc merged 92 commits intodevelopfrom
new.add-discusions-roomactionsview

Conversation

@gerzonc
Copy link
Contributor

@gerzonc gerzonc commented Jul 21, 2021

Proposed changes

Currently there is no way to see all the discussions created on a room. This PR achieve this by adding a Discussions action to RoomActionsView which shows all the existing Discussions in a room.

Issue(s)

Closes #3549

How to test or reproduce

In order to test this you should have a room (channel or group) with at least one discussion

  1. Go to the room
  2. Go to room actions

You should be able to see a Discussions item

Screenshots

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • Improvement (non-breaking change which improves a current function)
  • New feature (non-breaking change which adds functionality)
  • Documentation update (if none of the other choices apply)

Checklist

  • I have read the CONTRIBUTING doc
  • I have signed the CLA
  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works (if applicable)
  • I have added necessary documentation (if applicable)
  • Any dependent changes have been merged and published in downstream modules

Further comments

@lgtm-com
Copy link

lgtm-com bot commented Jul 21, 2021

This pull request introduces 1 alert when merging daba3b4 into 91f770c - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@gerzonc gerzonc marked this pull request as ready for review July 22, 2021 15:25
@AlexAlexandre AlexAlexandre self-requested a review July 28, 2021 20:04
@AlexAlexandre
Copy link
Contributor

@gerzonc, As discussed in a meeting, it will be necessary to include the discussion menu also in direct message users.

Let me know when I can review again.
thanks.

Copy link
Contributor

@AlexAlexandre AlexAlexandre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

);
};

export default DiscussionDetails;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should evaluate this later with UX team if we can reuse it on Message component, like we do with Threads.

@gerzonc gerzonc merged commit f0b5cd6 into develop Feb 1, 2022
@gerzonc gerzonc deleted the new.add-discusions-roomactionsview branch February 1, 2022 14:38
ivnxyz pushed a commit to NextiaDev/Rocket.Chat.ReactNative that referenced this pull request May 26, 2023
…-roomactionsview

[NEW] Add Discussions to RoomActionsView
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing discussion overview in channel menu

4 participants

Comments