Skip to content

Conversation

@juanultimate
Copy link
Contributor

  • Allow edit entry running
  • Enabling notifications when updating time entries
  • Refreshing data when creating/updating time entries
    image

@juanultimate juanultimate force-pushed the 333-allow-edit-entry-running branch from 3d1a37b to 1729f78 Compare June 11, 2020 06:06
@codecov
Copy link

codecov bot commented Jun 11, 2020

Codecov Report

Merging #375 into master will decrease coverage by 0.30%.
The diff coverage is 83.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #375      +/-   ##
==========================================
- Coverage   92.04%   91.74%   -0.31%     
==========================================
  Files          70       70              
  Lines        1157     1163       +6     
  Branches       72       74       +2     
==========================================
+ Hits         1065     1067       +2     
- Misses         68       71       +3     
- Partials       24       25       +1     
Impacted Files Coverage Δ
src/app/modules/time-clock/store/entry.effects.ts 45.00% <66.66%> (+0.73%) ⬆️
...ponents/details-fields/details-fields.component.ts 89.09% <87.50%> (-0.71%) ⬇️
...dules/time-entries/pages/time-entries.component.ts 100.00% <100.00%> (ø)
...modules/shared/components/clock/clock.component.ts 58.33% <0.00%> (-33.34%) ⬇️

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 c828171...a3907c6. Read the comment docs.

@juanultimate juanultimate force-pushed the 333-allow-edit-entry-running branch from 1729f78 to a3907c6 Compare June 11, 2020 13:30
@enriquezrene enriquezrene merged commit 974042d into master Jun 11, 2020
@enriquezrene enriquezrene deleted the 333-allow-edit-entry-running branch June 11, 2020 14:22
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