Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(auth): add provide #1704

Merged
merged 5 commits into from
Nov 12, 2023
Merged

feat(auth): add provide #1704

merged 5 commits into from
Nov 12, 2023

Conversation

cipchk
Copy link
Member

@cipchk cipchk commented Nov 12, 2023

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

Copy link

codecov bot commented Nov 12, 2023

Codecov Report

Merging #1704 (c159d04) into master (ae1cc30) will decrease coverage by 0.06%.
The diff coverage is 68.42%.

❗ Current head c159d04 differs from pull request most recent head 65af2e8. Consider uploading reports for the commit 65af2e8 to get more accurate results

@@            Coverage Diff             @@
##           master    #1704      +/-   ##
==========================================
- Coverage   95.14%   95.09%   -0.06%     
==========================================
  Files         315      315              
  Lines        8864     8882      +18     
  Branches     1661     1662       +1     
==========================================
+ Hits         8434     8446      +12     
- Misses        359      365       +6     
  Partials       71       71              
Files Coverage Δ
packages/auth/src/store/cookie-storage.service.ts 100.00% <ø> (ø)
packages/auth/src/store/local-storage.service.ts 100.00% <ø> (ø)
packages/auth/src/store/memory.service.ts 100.00% <ø> (ø)
packages/auth/src/store/session-storage.service.ts 100.00% <ø> (ø)
packages/auth/src/token/jwt/jwt.interceptor.ts 100.00% <ø> (ø)
...ckages/auth/src/token/simple/simple.interceptor.ts 100.00% <ø> (ø)
packages/auth/src/provide.ts 68.42% <68.42%> (ø)

@ng-alain-bot
Copy link
Contributor

ng-alain-bot commented Nov 12, 2023

Preview is ready!

@cipchk cipchk merged commit c0c731b into master Nov 12, 2023
9 checks passed
@cipchk cipchk deleted the auth-provide branch November 12, 2023 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants