Skip to content

Conversation

gqroot
Copy link

@gqroot gqroot commented Dec 14, 2020

No description provided.

@gqroot gqroot requested a review from scastillo-jp December 14, 2020 22:18
@codecov
Copy link

codecov bot commented Dec 14, 2020

Codecov Report

Merging #587 (906f48e) into master (13de42e) will increase coverage by 0.41%.
The diff coverage is 98.07%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #587      +/-   ##
==========================================
+ Coverage   93.09%   93.51%   +0.41%     
==========================================
  Files          85       92       +7     
  Lines        1564     1649      +85     
  Branches      107      107              
==========================================
+ Hits         1456     1542      +86     
- Misses         67       70       +3     
+ Partials       41       37       -4     
Impacted Files Coverage Δ
src/app/app-routing.module.ts 100.00% <ø> (ø)
...les/shared/components/sidebar/sidebar.component.ts 88.88% <ø> (ø)
...dules/time-entries/pages/time-entries.component.ts 83.33% <85.71%> (+0.52%) ⬆️
src/app/modules/users/store/user.reducers.ts 87.50% <87.50%> (ø)
src/app/guards/login-guard/login.guard.ts 100.00% <100.00%> (ø)
src/app/modules/login/login.component.ts 100.00% <100.00%> (+16.66%) ⬆️
...app/modules/login/services/azure.ad.b2c.service.ts 86.95% <100.00%> (+4.60%) ⬆️
...ponents/details-fields/details-fields.component.ts 89.61% <100.00%> (+2.43%) ⬆️
.../components/entry-fields/entry-fields.component.ts 98.48% <100.00%> (-1.52%) ⬇️
...project-list-hover/project-list-hover.component.ts 88.63% <100.00%> (+0.26%) ⬆️
... and 15 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 0601f5b...906f48e. Read the comment docs.

@gqroot gqroot merged commit 79fadf5 into master Dec 14, 2020
@gqroot gqroot deleted the TT-78_CookieExpiration branch December 14, 2020 22:24
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