Skip to content

Conversation

VanessaIniguez
Copy link
Collaborator

Have the data persist in the modal when the modal is closed.

@codecov
Copy link

codecov bot commented Feb 24, 2021

Codecov Report

Merging #639 (0a78920) into master (13de42e) will increase coverage by 1.14%.
The diff coverage is 97.56%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #639      +/-   ##
==========================================
+ Coverage   93.09%   94.24%   +1.14%     
==========================================
  Files          85       95      +10     
  Lines        1564     1789     +225     
  Branches      107      119      +12     
==========================================
+ Hits         1456     1686     +230     
+ Misses         67       65       -2     
+ Partials       41       38       -3     
Impacted Files Coverage Δ
src/app/app-routing.module.ts 100.00% <ø> (ø)
...omponents/customer-list/customer-list.component.ts 92.10% <ø> (ø)
...time-entries-table/time-entries-table.component.ts 55.55% <ø> (ø)
...nents/time-range-form/time-range-form.component.ts 100.00% <ø> (ø)
...dules/time-entries/pages/time-entries.component.ts 85.13% <81.81%> (+2.32%) ⬆️
...ponents/details-fields/details-fields.component.ts 93.47% <94.28%> (+6.29%) ⬆️
src/app/modules/users/store/user.reducers.ts 94.73% <94.73%> (ø)
src/app/guards/login-guard/login.guard.ts 100.00% <100.00%> (ø)
...nologies-report-guard/technologies-report.guard.ts 100.00% <100.00%> (ø)
src/app/modules/login/login.component.ts 100.00% <100.00%> (+16.66%) ⬆️
... and 35 more

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 d5773d7...8dc0d4a. Read the comment docs.

…her it is a new entry or an existing one and persist the data in the form, create their respective tests
@scastillo-jp scastillo-jp merged commit 3dc840d into master Mar 1, 2021
@scastillo-jp scastillo-jp deleted the TT-141-check-overlap-hours branch March 1, 2021 15:56
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.

5 participants