Skip to content

Comments

[NEW] Message Forwarding Init#25939

Merged
dougfabris merged 8 commits intoRocketChat:new/message-forwardingfrom
nishant23122000:message-forwarding
Jul 4, 2022
Merged

[NEW] Message Forwarding Init#25939
dougfabris merged 8 commits intoRocketChat:new/message-forwardingfrom
nishant23122000:message-forwarding

Conversation

@nishant23122000
Copy link
Contributor

@nishant23122000 nishant23122000 commented Jun 21, 2022

Proposed changes (including videos or screenshots)

  1. Added Share Message imperative Modal
  2. Added Composer by importing from ee folder.

Forwarding Modal

  • Multiple Selection
  • Message composer
  • Preview
  • Send Action and Copy Action

Issue(s)

Steps to test or reproduce

Further comments

@nishant23122000 nishant23122000 requested review from a team as code owners June 21, 2022 04:28
@nishant23122000 nishant23122000 changed the base branch from new/message-forwarding to develop June 21, 2022 04:28
@nishant23122000 nishant23122000 changed the base branch from develop to new/message-forwarding June 21, 2022 04:29
@nishant23122000 nishant23122000 marked this pull request as draft June 21, 2022 04:33
@lgtm-com
Copy link

lgtm-com bot commented Jun 21, 2022

This pull request introduces 5 alerts and fixes 1 when merging 6758cca into 5a37518 - view on LGTM.com

new alerts:

  • 4 for Information exposure through a stack trace
  • 1 for Missing rate limiting

fixed alerts:

  • 1 for Missing rate limiting

@ankar84
Copy link

ankar84 commented Jun 21, 2022

If I get header of that PR right it could be related to RocketChat/feature-requests#308
And a very useful and waited feature!

@dougfabris
Copy link
Member

You're right @ankar84 😄
@nishant23122000 is the contributor in charge of this waited feature
Soon we'll have this our spaceship!

@dougfabris dougfabris marked this pull request as ready for review June 27, 2022 13:22
@dougfabris dougfabris merged commit 6f914f8 into RocketChat:new/message-forwarding Jul 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants