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.

Sign up for GitHub

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

Jump to bottom

Added app.json, to allow for one click installation in Heroku #9

Open
wants to merge 2 commits into
base: latest
Choose a base branch
from
Open

Added app.json, to allow for one click installation in Heroku #9

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
81b4888
Added app.json, to allow for one click installation in Heroku
dvidsilva Oct 22, 2016
46d1f2a
Improvements to team_join event
dvidsilva Oct 26, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,3 +65,11 @@ into both of those. `#admin` is where `/admin` commands will go.
* Grant the user admin privileges
* Log in with the user on your browser (by visiting <yourslack>.slack.com)
* Visit https://<appname>.herokuapp.com/install-inviter?team=<team_id> -- the URL will be displayed the first time you try to accept a user anyway.

### Welcome message to new users

The `WELCOME_MESSAGE_TEXT` config var controls what message is sent to the new users when they join. By default it says:
`Welcome! Review the ${cocTxt} before participating in the community.`

To disable the welcome message, set the config var `SEND_WELCOME_MESSAGE` to `FALSE`.

23 changes: 23 additions & 0 deletions app.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
{
"name": "Wheelie Slack App",
"description": "Wheelie is a Slack App originally written for WeAllJS.com.",
"repository": "https://github.com/WeAllJS/wheelie-slack-app",
"keywords": ["node", "express", "slack", "bot"],
"env": {
"SEND_WELCOME_MESSAGE": {
"description": "If the value of is 'FALSE', no message will be sent to new members when they join "
},
"WELCOME_MESSAGE_TEXT": {
"description": "A message that will be send to new members once they join the group"
},
"SLACK_CLIENT_ID" : {
"description": ""
},
"SLACK_CLIENT_SECRET" : {
"description": ""
},
"VERIFICATION_TOKEN" : {
"description": ""
}
}
}
Copy link
Member

Choose a reason for hiding this comment

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

How do you feel about filling in these descriptions?

Copy link
Author

Choose a reason for hiding this comment

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

I don't know what exactly goes there, would you mind helping with that text, I don't wanna confuse the users.

Copy link
Author

Choose a reason for hiding this comment

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

I can separate the things into two different PRs too, they're not really the same feature.

Copy link
Member

Choose a reason for hiding this comment

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

They're basically application-level tokens. This isn't really a blocker either so don't worry about it. They're used by Slack to identify and authenticate applications, and there verification token is for authenticating individual requests -- these are simply things you grab from slack

20 changes: 19 additions & 1 deletion lib/events/team_join.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,22 @@
const slack = require('slack')
const bluebird = require('bluebird')
const chat = bluebird.promisifyAll(slack.chat)

const cocTxt = process.env.COC_URL ? `<${process.env.COC_URL}|Code of Conduct>` : 'Code of Conduct'
const welcome_message_text = process.env.WELCOME_MESSAGE_TEXT || `Welcome! Review the ${cocTxt} before participating in the community.`
const send_welcome_message = process.env.SEND_WELCOME_MESSAGE === 'FALSE' ? false : true
Copy link
Member

Choose a reason for hiding this comment

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

We need to figure out a way to make this configurable per-Slack, by Admins. I've been considering adding an admin-only /config command. How do you feel about this part of your patch being left unused until it can be configured? Different Slacks are going to have completely different needs as far as their welcome messages go, too. >_<

Copy link
Author

Choose a reason for hiding this comment

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

Yep, I realized that later, this would be global config for everyone? how are you currently managing configurations per team, I can change it so it uses that.

Copy link
Member

Choose a reason for hiding this comment

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

We don't have a way to do that right now, which kinda sucks but was in my mental to-do list for Soon™.

You did the bulk of the work for this feature too so what I meant is that we could merge this with default messages, and then customize it once the feature is up. I'm still not sure what the best way to do that is.


module.exports = (data, cb) => {
console.log('new user joined team: ', data)
cb()
if (!send_welcome_message || !welcome_message_text) {
return cb()
}
chat.postMessageAsync({
token: data.token,
channel: data.user.id,
text: welcome_message_text
}).then(() => {
return cb()
})
}

4 changes: 4 additions & 0 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,5 +20,9 @@
},
"engines": {
"node": "5.9.1"
},
"devDependencies": {
"eslint": "^3.8.1",
"eslint-plugin-react": "^6.4.1"
}
}