Skip to content

Comments

changes to get reacted message information#33

Open
songupta7 wants to merge 4 commits intoExpediaGroup:masterfrom
songupta7:task-get-reaction-message-info
Open

changes to get reacted message information#33
songupta7 wants to merge 4 commits intoExpediaGroup:masterfrom
songupta7:task-get-reaction-message-info

Conversation

@songupta7
Copy link
Contributor

@songupta7 songupta7 commented May 12, 2022

This PR is continuation with PR #28 .
I am requesting review for the code changes done to implement GetReactionMessageInfo Command from the flyte

@songupta7 songupta7 requested a review from a team as a code owner May 12, 2022 12:43
@songupta7 songupta7 marked this pull request as draft May 12, 2022 12:44
@songupta7
Copy link
Contributor Author

@pamelin Please review this PR so that I can close the PR-28.

@songupta7
Copy link
Contributor Author

@pamelin Waiting for your valuable inputs , Please review when you get sometime...

@songupta7
Copy link
Contributor Author

@pamelin Can you please review this PR. please share the ETA for closure so that we can plan our tasks.
and DEA-4745 . As we created 2 PR's of single , which was earlier tracked as DEA-4745. if in case you require another ticket, I have created one for you. DEA-5954 .
thanks & regards

@pamelin
Copy link
Contributor

pamelin commented Jun 6, 2022

@pamelin Can you please review this PR. please share the ETA for closure so that we can plan our tasks. and DEA-4745 . As we created 2 PR's of single , which was earlier tracked as DEA-4745. if in case you require another ticket, I have created one for you. DEA-5954 . thanks & regards

Hey @songupta7 ,
The details provided in this comment do not mean much for the open source project. Please try to avoid adding any details which do not mean much in the oss projects.

FYI currently there are not many maintainers of this project. I myself was away for a while. I will try to look at this as soon as I can.

@songupta7
Copy link
Contributor Author

songupta7 commented Jun 6, 2022

@pamelin Can you please review this PR. please share the ETA for closure so that we can plan our tasks. and DEA-4745 . As we created 2 PR's of single , which was earlier tracked as DEA-4745. if in case you require another ticket, I have created one for you. DEA-5954 . thanks & regards

Hey @songupta7 , The details provided in this comment do not mean much for the open source project. Please try to avoid adding any details which do not mean much in the oss projects.

FYI currently there are not many maintainers of this project. I myself was away for a while. I will try to look at this as soon as I can.

Apologies, I can delete the comment, If you think, that's ok.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants