Skip to content

Conversation

DiegoTinitana
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented May 26, 2020

Codecov Report

Merging #303 into master will decrease coverage by 0.16%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #303      +/-   ##
==========================================
- Coverage   94.15%   93.99%   -0.17%     
==========================================
  Files          63       63              
  Lines         976      966      -10     
  Branches       59       55       -4     
==========================================
- Hits          919      908      -11     
+ Misses         37       36       -1     
- Partials       20       22       +2     
Impacted Files Coverage Δ
.../components/month-picker/month-picker.component.ts 100.00% <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%> (ø)
...dules/time-entries/pages/time-entries.component.ts 100.00% <100.00%> (ø)
...ponents/details-fields/details-fields.component.ts 90.19% <0.00%> (-2.91%) ⬇️

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 7133d69...6bb89b2. Read the comment docs.

@@ -39,8 +31,9 @@ export class TimeEntriesComponent implements OnInit {
}

editEntry(entryId: string) {
console.log(this.dataByMonth);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's remove this line.

@enriquezrene enriquezrene merged commit baa2cb9 into master May 27, 2020
@enriquezrene enriquezrene deleted the 180_Show_time-entries_for_selected_month branch May 27, 2020 02:05
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