[BUG] IPostMessageReacted returns field named `isRemoved` instead of `isReacted`
Description:
-
AppsEngine have
IPostMessageReacted, Handler for after a message get a reaction or a reaction got removed . has methodexecutePostMessageReactedwhich takes one of the parameter of typeIMessageReactionContextwhich includes isReacted field to know whether the emoji has been added or deleted. But everytime if we react with emoji in messages instead of having abooleanvalue in isReacted field, we are gettingundefinedregardless the operation we are making ( add or delete ). -
Reason for having value as
undefinedeverytime is from RocketChat, we are returning the field namedisRemovedinstead ofisReacted. Have a look over here. But Expected field should have nameisReactedas per the docs.
Steps to reproduce:
- Implement
IPostMessageReactedin RocketChat App. - log
isReactedfield in console. - Deploy the app and React any message with emoji and check console.
OR
- Set up this playground app to avoid the step 1 and follow rest of steps.
Expected behavior:
- Instead of having
undefinedinisReactedwe are expecting it to bebooleanvalue as to know the emoji operation within message (add or delete).
Actual behavior:
- As we can see, getting isReacted field as
undefined
Note:
- [x] willing to work on this.
Working on it🚀.
Thanks @Nabhag8848 ! That's a nice catch!