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

Allow military time with colon to be used for scheduling standups #159

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

Conversation

billneff79
Copy link

e.g. schedule standup 13:00 will schedule it for 1:00 pm. Using am/pm still works - this is just added functionality

@billneff79 billneff79 changed the title Allow military time to be used for scheduling standups Allow military time with colon to be used for scheduling standups Sep 4, 2019
@codecov-io
Copy link

Codecov Report

Merging #159 into develop will decrease coverage by <.01%.
The diff coverage is 83.33%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #159      +/-   ##
===========================================
- Coverage    86.44%   86.43%   -0.01%     
===========================================
  Files           58       59       +1     
  Lines         1372     1379       +7     
  Branches       181      182       +1     
===========================================
+ Hits          1186     1192       +6     
- Misses         186      187       +1
Impacted Files Coverage Δ
lib/helpers/time.js 98.43% <100%> (ø) ⬆️
features/steps/helper-time.js 88.09% <80%> (-1.1%) ⬇️
models/index.js 92.85% <0%> (ø)
lib/helpers/reports/transformers/index.js 100% <0%> (+7.14%) ⬆️

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 4c8b97a...192617a. Read the comment docs.

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