-
Notifications
You must be signed in to change notification settings - Fork 2.7k
MM-10516: Added interactive action support to ephemeral messages #2351
Conversation
- Preserve the cookie in PostActions - Replaced doPostAction with doPostActionWithCookie
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Lev!
Approving changes as-is. As noted on the server PR, we would test with the Giphy plugin once this PR has been merged (along with Redux). So I will do testing there and provide client-side feedback there.
I can also test the Giphy plugin on mobile to see if a mobile PR needs to be submitted.
How can I do a "E2E" on this feature? Does updating the redux hash in |
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks @levb for the hint. 👍 Got my local setup working. |
…termost#2351) * MM-10516: Added interactive action support to ephemeral messages - Preserve the cookie in PostActions - Replaced doPostAction with doPostActionWithCookie * Updated mattermost-redux include to the supporting changes. mattermost/mattermost-redux@aa1b7f5
Summary
This PR supports adding interactive actions to ephemeral messages. The client now recognizes "action cookies" that are sent by mattermost-server included in PostAction's; It stores them and sends them back to the server when submitting DoPostActionRequest.
Ticket Link
https://mattermost.atlassian.net/browse/MM-10516
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
make check-style
to check for style errors (required for all pull requests)make test
to ensure unit and component tests passedSee Also: