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/5733 Add private note action in macros #5805
Conversation
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.
Looks good. Added a small update to resolve the notes in preview.
Tested locally - https://www.loom.com/share/2978fa2c6e434974b5a4d94ea791aef2
def add_private_note(message) | ||
return if conversation_a_tweet? | ||
|
||
params = { content: message[0], private: true } | ||
|
||
# Added reload here to ensure conversation us persistent with the latest updates | ||
mb = Messages::MessageBuilder.new(@user, @conversation.reload, params) | ||
mb.perform | ||
end |
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.
@tejaswinichile Couldn't this be a common function for both send_message and private note?
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Fixes #5733
Type of change
How Has This Been Tested?
Tested on local, can save the macros with private note and when we execute the macros able to see private note in the conversation
Checklist: