Skip to content

Conversation

juanultimate
Copy link
Contributor

This PR fixes how the data for reports is queried. Now the start date begins at 00:00 and the end date finish at 23:59.

image

@codecov
Copy link

codecov bot commented Jun 15, 2020

Codecov Report

Merging #381 into master will decrease coverage by 0.42%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #381      +/-   ##
==========================================
- Coverage   92.01%   91.58%   -0.43%     
==========================================
  Files          71       71              
  Lines        1177     1177              
  Branches       75       75              
==========================================
- Hits         1083     1078       -5     
- Misses         69       74       +5     
  Partials       25       25              
Impacted Files Coverage Δ
...nents/time-range-form/time-range-form.component.ts 100.00% <ø> (ø)
...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.effects.ts 45.45% <100.00%> (ø)
...modules/shared/components/clock/clock.component.ts 58.33% <0.00%> (-41.67%) ⬇️

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 8394861...c861258. Read the comment docs.

@enriquezrene enriquezrene merged commit 42e1df4 into master Jun 15, 2020
@enriquezrene enriquezrene deleted the 346-fixing-report-time branch June 15, 2020 13:32
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