Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

Slack fixes #66

Merged
merged 7 commits into from
Feb 8, 2017
Merged

Slack fixes #66

merged 7 commits into from
Feb 8, 2017

Conversation

uttpal
Copy link
Contributor

@uttpal uttpal commented Feb 8, 2017

fixed #62 #63 #65 #45

Copy link

@parthshah parthshah left a comment

Choose a reason for hiding this comment

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

Looks good, just a few minor changes

},
{
title: 'Project Type',
value: data.project.type,

Choose a reason for hiding this comment

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

Please map raw project types to the following -

  • app_dev --> Full App
  • generic --> Work Project
  • visual_prototype --> Design & Prototype
  • Visual Design --> Design

short: false,
},
],
};
},
projectUnclaimed: (data) => {
return {
icon_url: 'https://emoji.slack-edge.com/T03R80JP7/coder-the-bot/85ae574c0c7063ef.png',

Choose a reason for hiding this comment

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

please move to constants file, 'CoderBotIcon'

fields: [
{
title: 'Project Type',
value: data.project.type,

Choose a reason for hiding this comment

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

See above, thanks!

},
projectUnclaimedReposted: (data) => {
return {
icon_url: 'https://emoji.slack-edge.com/T03R80JP7/coder-error/cd2633216e7fd385.png',

Choose a reason for hiding this comment

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

Move to constants CoderErrorIcon

fields: [
{
title: 'Project Type',
value: data.project.type,

Choose a reason for hiding this comment

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

Same as above

},
projectClaimed: (data) => {
return {
icon_url: 'https://emoji.slack-edge.com/T03R80JP7/coder-grinning/a3b7f3fe9e838377.png',

Choose a reason for hiding this comment

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

Move to constants - CoderGrinningIcon

@parthshah parthshah merged commit 0b8e801 into dev Feb 8, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Slackbot notification when project has been claimed by copilot
2 participants