Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Message Forwarding #26662

Merged
merged 41 commits into from
Mar 15, 2023
Merged

feat: Message Forwarding #26662

merged 41 commits into from
Mar 15, 2023

Conversation

dougfabris
Copy link
Member

@dougfabris dougfabris commented Aug 23, 2022

Proposed changes (including videos or screenshots)

Implement the ability to forward messages to another channel, thread, or discussion. This is a high demand capability that is currently missing in Rocket.Chat. It can significantly improve the user's experience and her/his productivity. The scope of this project will be limited to the web application (not for mobile apps).

Special thanks to @nishant23122000 that helped us with this during GSOC 2022

Issue(s)

Fixes #396
Closes #11179

Steps to test or reproduce

Further comments

nishant23122000 and others added 6 commits July 4, 2022 20:29
* share message intialization

* lint fix

* condition before showing

* small opt

* mod:change to Message component for general solution

* update: react-hook-form;removed any type

* typo error

Co-authored-by: Douglas Fabris <[email protected]>
@ankar84
Copy link

ankar84 commented Aug 24, 2022

Wait for that feature a lot!

@lgtm-com
Copy link

lgtm-com bot commented Dec 2, 2022

This pull request introduces 1 alert when merging 0a14428 into 07ea6d1 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@dougfabris dougfabris added this to the 6.0.0-prep milestone Dec 19, 2022
@dougfabris dougfabris marked this pull request as ready for review January 27, 2023 18:23
@dougfabris dougfabris requested review from a team as code owners January 27, 2023 18:23
ggazzo
ggazzo previously approved these changes Jan 27, 2023
@dougfabris dougfabris added this to the 6.1.0 milestone Feb 28, 2023
@dougfabris dougfabris requested a review from ggazzo February 28, 2023 15:32
@dougfabris dougfabris changed the title [NEW] Message Forwarding feat: Message Forwarding Mar 13, 2023
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Mar 15, 2023
@dougfabris dougfabris requested a review from ggazzo March 15, 2023 21:19
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Mar 15, 2023
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Mar 15, 2023
@ggazzo ggazzo merged commit c018be1 into develop Mar 15, 2023
@ggazzo ggazzo deleted the new/message-forwarding branch March 15, 2023 22:34
@casalsgh casalsgh modified the milestones: 6.1.0, 6.2.0 Apr 12, 2023
@sampaiodiego sampaiodiego mentioned this pull request May 16, 2023
@VergeDX
Copy link

VergeDX commented Apr 10, 2024

How about multiple message forwarding?
Is there any plan...?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stat: ready to merge PR tested and approved waiting for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ability forward messages into other chat rooms
6 participants