Skip to content
This repository has been archived by the owner on Nov 14, 2019. It is now read-only.

[WIP] Slack app #126

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

[WIP] Slack app #126

wants to merge 37 commits into from

Conversation

mgwalker
Copy link
Member

@mgwalker mgwalker commented Jan 6, 2017

Make the bot Slack app compatible, add buttons and stuff

Other goals:

  • Ensure the bot can still work as a standalone bot ("custom integration") as well as a Slack app.
  • Document how to set the bot up as an app

@codecov-io
Copy link

codecov-io commented Jan 6, 2017

Codecov Report

Merging #126 into develop will decrease coverage by -4.66%.
The diff coverage is 48.68%.

@@            Coverage Diff             @@
##           develop    #126      +/-   ##
==========================================
- Coverage    84.85%   80.2%   -4.66%     
==========================================
  Files           56      65       +9     
  Lines         1321    1450     +129     
  Branches       174     206      +32     
==========================================
+ Hits          1121    1163      +42     
- Misses         200     287      +87
Impacted Files Coverage Δ
lib/helpers/index.js 100% <ø> (ø)
lib/bot/index.js 100% <ø> (ø)
lib/bot/userReport.js 84% <100%> (+0.66%)
lib/bot/message-builders/standup-config.js 14.81% <14.81%> (ø)
lib/bot/interactive/index.js 15.38% <15.38%> (ø)
lib/bot/interactive/standup-config.js 17.85% <17.85%> (ø)
lib/bot/interactive/reminder-set-ooo.js 28.57% <28.57%> (ø)
lib/bot/interactive/slash-standup.js 33.33% <33.33%> (ø)
lib/helpers/doInterview.js 64.58% <33.33%> (+8.06%)
lib/bot/interactive/reminder-start-interview.js 50% <50%> (ø)
... and 16 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4b3b6d2...0f2bfce. Read the comment docs.

…lsewhere; refactor doInterview to use the new standup report saver; update Standup model and migrations to add a unique index to Standup so we can upsert it
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.

2 participants