Skip to content

Conversation

@juanultimate
Copy link
Contributor

Allow users to restart a time entry.

@juanultimate juanultimate requested review from DiegoTinitana and enriquezrene and removed request for enriquezrene June 23, 2020 05:28
@juanultimate juanultimate changed the title Allowing restart time entry Allowing users to restart time entries Jun 23, 2020
@codecov
Copy link

codecov bot commented Jun 23, 2020

Codecov Report

Merging #412 into master will increase coverage by 0.24%.
The diff coverage is 93.97%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #412      +/-   ##
==========================================
+ Coverage   90.27%   90.52%   +0.24%     
==========================================
  Files          72       72              
  Lines        1234     1266      +32     
  Branches       78       81       +3     
==========================================
+ Hits         1114     1146      +32     
- Misses         92       93       +1     
+ Partials       28       27       -1     
Impacted Files Coverage Δ
src/app/modules/time-clock/store/entry.effects.ts 46.75% <58.33%> (+1.29%) ⬆️
...ponents/details-fields/details-fields.component.ts 92.06% <100.00%> (+1.74%) ⬆️
...project-list-hover/project-list-hover.component.ts 85.71% <100.00%> (-0.40%) ⬇️
...c/app/modules/time-clock/services/entry.service.ts 100.00% <100.00%> (ø)
src/app/modules/time-clock/store/entry.actions.ts 100.00% <100.00%> (ø)
src/app/modules/time-clock/store/entry.reducer.ts 97.61% <100.00%> (+0.11%) ⬆️
...dules/time-entries/pages/time-entries.component.ts 90.47% <100.00%> (+0.47%) ⬆️
...modules/shared/components/clock/clock.component.ts 91.66% <0.00%> (+33.33%) ⬆️

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 6e9edb1...c9edc24. Read the comment docs.

@juanultimate juanultimate force-pushed the 385-fixing-editing-current-time-entry branch from 955d014 to c9edc24 Compare June 23, 2020 14:40
@enriquezrene enriquezrene merged commit 9c27fcb into master Jun 23, 2020
@enriquezrene enriquezrene deleted the 385-fixing-editing-current-time-entry branch June 23, 2020 16:07
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