Skip to content

Conversation

enriquezrene
Copy link
Contributor

Clock out functionality

@codecov
Copy link

codecov bot commented Apr 28, 2020

Codecov Report

Merging #190 into master will increase coverage by 0.29%.
The diff coverage is 79.16%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #190      +/-   ##
==========================================
+ Coverage   91.51%   91.80%   +0.29%     
==========================================
  Files          59       59              
  Lines         872      830      -42     
  Branches       54       50       -4     
==========================================
- Hits          798      762      -36     
+ Misses         53       46       -7     
- Partials       21       22       +1     
Impacted Files Coverage Δ
...modules/shared/components/modal/modal.component.ts 100.00% <ø> (ø)
...project-list-hover/project-list-hover.component.ts 78.26% <54.54%> (-7.46%) ⬇️
...rc/app/modules/time-clock/store/entry.selectors.ts 70.00% <60.00%> (-5.00%) ⬇️
.../components/entry-fields/entry-fields.component.ts 90.47% <80.00%> (-2.21%) ⬇️
...p/modules/time-clock/pages/time-clock.component.ts 84.61% <81.81%> (-4.28%) ⬇️
...c/app/modules/time-clock/services/entry.service.ts 100.00% <100.00%> (+42.85%) ⬆️
src/app/modules/time-clock/store/entry.actions.ts 100.00% <100.00%> (ø)
src/app/modules/time-clock/store/entry.reducer.ts 82.35% <100.00%> (+2.35%) ⬆️
...dules/time-entries/pages/time-entries.component.ts 100.00% <100.00%> (ø)
... and 1 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 27011b4...bfcd3ae. Read the comment docs.

@enriquezrene enriquezrene merged commit 3acd1e8 into master Apr 28, 2020
@enriquezrene enriquezrene deleted the 151-clock-out-time-entries branch April 28, 2020 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants