-
Notifications
You must be signed in to change notification settings - Fork 5
Conversation
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 good, just a few minor changes
src/common/constants.js
Outdated
}, | ||
{ | ||
title: 'Project Type', | ||
value: data.project.type, |
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.
Please map raw project types to the following -
- app_dev --> Full App
- generic --> Work Project
- visual_prototype --> Design & Prototype
- Visual Design --> Design
src/common/constants.js
Outdated
short: false, | ||
}, | ||
], | ||
}; | ||
}, | ||
projectUnclaimed: (data) => { | ||
return { | ||
icon_url: 'https://emoji.slack-edge.com/T03R80JP7/coder-the-bot/85ae574c0c7063ef.png', |
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.
please move to constants file, 'CoderBotIcon'
src/common/constants.js
Outdated
fields: [ | ||
{ | ||
title: 'Project Type', | ||
value: data.project.type, |
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.
See above, thanks!
src/common/constants.js
Outdated
}, | ||
projectUnclaimedReposted: (data) => { | ||
return { | ||
icon_url: 'https://emoji.slack-edge.com/T03R80JP7/coder-error/cd2633216e7fd385.png', |
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.
Move to constants CoderErrorIcon
src/common/constants.js
Outdated
fields: [ | ||
{ | ||
title: 'Project Type', | ||
value: data.project.type, |
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.
Same as above
src/common/constants.js
Outdated
}, | ||
projectClaimed: (data) => { | ||
return { | ||
icon_url: 'https://emoji.slack-edge.com/T03R80JP7/coder-grinning/a3b7f3fe9e838377.png', |
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.
Move to constants - CoderGrinningIcon
fixed #62 #63 #65 #45