-
Notifications
You must be signed in to change notification settings - Fork 84
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
#228 - Basic Hello World Slack Command Bot #236
#228 - Basic Hello World Slack Command Bot #236
Conversation
@Haimantika Merging this PR closes issue #228 |
@Haimantika any updates ? |
I have assigned my team mate. You can also join us for PR reviews tomorrow on Discord and get it reviewed live. |
@loks0n Any Updates ? |
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.
Very solid PR ✨ I left some comments to ensure we follow our typical approach to templates. Please address and Ill give it another look
Co-authored-by: Matej Bačo <[email protected]>
Co-authored-by: Matej Bačo <[email protected]>
@Meldiron I will do the changes and let you know. |
@loks0n Have done the changes |
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 great, a few final things
@loks0n have done the changes |
@Haimantika Am i Eligible for Appwrite Swags ? |
|
Don't worry - you are eligible for the swags 🥳 We will merge the PR when Meldiron is able to review. |
@ManasMadan add your discord handle here so we can contact you about swags after the PR is merged soon :) |
@tessamero My discord username is manasmadan |
What does this PR do?
This PR Closes the issue #228 by implementing a Slack Command Bot Template in Node.js
Test Plan
Demo Video Link
Related PRs and Issues
This PR is related to issue #228 and merging this PR closes that issue
Have you read the Contributing Guidelines on issues?
Yes I Have read the contributing guidelines