Skip to content

Conversation

@jorgecod
Copy link
Contributor

Issue 150

  • Get active entry
  • Update active entry

@codecov
Copy link

codecov bot commented Apr 27, 2020

Codecov Report

Merging #188 into master will decrease coverage by 0.39%.
The diff coverage is 85.54%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #188      +/-   ##
==========================================
- Coverage   91.90%   91.51%   -0.40%     
==========================================
  Files          57       59       +2     
  Lines         791      872      +81     
  Branches       46       54       +8     
==========================================
+ Hits          727      798      +71     
- Misses         47       53       +6     
- Partials       17       21       +4     
Impacted Files Coverage Δ
...info/components/create-customer/create-customer.ts 89.74% <ø> (ø)
...c/app/modules/time-clock/services/entry.service.ts 57.14% <0.00%> (-42.86%) ⬇️
...project-list-hover/project-list-hover.component.ts 85.71% <50.00%> (-14.29%) ⬇️
...rc/app/modules/time-clock/store/entry.selectors.ts 75.00% <75.00%> (ø)
src/app/modules/time-clock/store/entry.reducer.ts 80.00% <87.50%> (+17.50%) ⬆️
.../components/entry-fields/entry-fields.component.ts 92.68% <92.68%> (ø)
src/app/modules/time-clock/store/entry.actions.ts 100.00% <100.00%> (ø)
...p/modules/time-clock/pages/time-clock.component.ts 88.88% <0.00%> (+1.58%) ⬆️

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 16cf3a3...3290e05. Read the comment docs.

@enriquezrene enriquezrene force-pushed the 150/Update-current-time-entry branch from 38e0b1b to 3a22b1d Compare April 27, 2020 16:36
@enriquezrene enriquezrene merged commit 6416f1a into master Apr 27, 2020
@enriquezrene enriquezrene deleted the 150/Update-current-time-entry branch April 27, 2020 16:40
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.

3 participants