Skip to content
This repository was archived by the owner on Aug 22, 2025. It is now read-only.

Specify allowed domains#81

Merged
codedge merged 5 commits intomasterfrom
allowed-domains
Jun 15, 2021
Merged

Specify allowed domains#81
codedge merged 5 commits intomasterfrom
allowed-domains

Conversation

@codedge
Copy link
Copy Markdown
Owner

@codedge codedge commented Jun 12, 2021

This PR addresses #78

You can now specify the domains allowed that should receive a magic link.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jun 15, 2021

Codecov Report

Merging #81 (de12705) into master (58748b2) will increase coverage by 1.42%.
The diff coverage is 90.90%.

❗ Current head de12705 differs from pull request most recent head 48ef13f. Consider uploading reports for the commit 48ef13f to get more accurate results
Impacted file tree graph

@@             Coverage Diff              @@
##             master      #81      +/-   ##
============================================
+ Coverage     72.50%   73.92%   +1.42%     
- Complexity       67       73       +6     
============================================
  Files            14       14              
  Lines           240      257      +17     
============================================
+ Hits            174      190      +16     
- Misses           66       67       +1     
Flag Coverage Δ
unittests 73.92% <90.90%> (+1.42%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/MagicLinkManager.php 92.50% <83.33%> (-4.06%) ⬇️
src/Http/Controllers/Cp/SettingsController.php 100.00% <100.00%> (ø)
src/Http/Controllers/MagicLinkController.php 100.00% <100.00%> (+4.34%) ⬆️
src/Repositories/SettingsRepository.php 100.00% <100.00%> (ø)
src/ServiceProvider.php 100.00% <100.00%> (ø)

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 58748b2...48ef13f. Read the comment docs.

@codedge codedge merged commit 2668729 into master Jun 15, 2021
@codedge codedge deleted the allowed-domains branch June 15, 2021 21:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants