Skip to content
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

context command to pin messages in respective thread #1188 #1194

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

andrew-09
Copy link
Contributor

#1188

Will give a warning message when the message is not in a thread, the user invoker is not the owner of the thread, or the number of pinned answers exceeds 10

image

image

@andrew-09 andrew-09 requested a review from a team as a code owner November 3, 2024 21:11
…ages in thread. Added additional check to see if in the 'questions' channel along with config parameter in PinAnswerCommand
andrew-09 added 2 commits November 9, 2024 11:41
…ary failure case. Update a reply for more clarity for users
@andrew-09 andrew-09 requested a review from Taz03 November 9, 2024 19:42
Copy link
Member

@ankitsmt211 ankitsmt211 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good work 👍

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