Skip to content

Conversation

enriquezrene
Copy link
Contributor

fix: #205 adding checks to restrict access for admin and non-admin users

@codecov
Copy link

codecov bot commented May 29, 2020

Codecov Report

Merging #307 into master will increase coverage by 0.51%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #307      +/-   ##
==========================================
+ Coverage   93.99%   94.51%   +0.51%     
==========================================
  Files          63       64       +1     
  Lines         966      984      +18     
  Branches       55       59       +4     
==========================================
+ Hits          908      930      +22     
+ Misses         36       32       -4     
  Partials       22       22              
Impacted Files Coverage Δ
src/app/app-routing.module.ts 100.00% <ø> (ø)
src/app/guards/admin-guard/admin-guard.ts 100.00% <100.00%> (ø)
src/app/guards/login-guard/login.guard.ts 100.00% <100.00%> (ø)
...app/modules/login/services/azure.ad.b2c.service.ts 93.33% <100.00%> (+0.47%) ⬆️
...les/shared/components/sidebar/sidebar.component.ts 100.00% <100.00%> (ø)
...dules/time-entries/pages/time-entries.component.ts 100.00% <0.00%> (ø)
...modules/shared/components/clock/clock.component.ts 91.66% <0.00%> (+33.33%) ⬆️

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 27bc5a0...8557dcc. Read the comment docs.

@enriquezrene enriquezrene merged commit 2a04181 into master May 29, 2020
@enriquezrene enriquezrene deleted the 297-add-extra-features-to-projects-table branch May 29, 2020 17:58
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.

Show/Hide menu options based on role values
1 participant