Skip to content
This repository has been archived by the owner on Sep 13, 2022. It is now read-only.

Prompt for authentication to confirm matching with existing user on initial LTI launch (fix #36) #47

Merged
merged 7 commits into from
May 2, 2019

Conversation

dac514
Copy link
Contributor

@dac514 dac514 commented May 2, 2019

No description provided.

@codecov
Copy link

codecov bot commented May 2, 2019

Codecov Report

Merging #47 into dev will increase coverage by 0.12%.
The diff coverage is 79.62%.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev      #47      +/-   ##
============================================
+ Coverage     73.42%   73.55%   +0.12%     
- Complexity      281      292      +11     
============================================
  Files             9        9              
  Lines           967     1002      +35     
============================================
+ Hits            710      737      +27     
- Misses          257      265       +8
Impacted Files Coverage Δ Complexity Δ
inc/namespace.php 91.52% <100%> (+1.72%) 0 <0> (ø) ⬇️
inc/class-admin.php 81.74% <100%> (+0.07%) 80 <0> (+1) ⬆️
inc/class-controller.php 43.1% <55%> (-2.73%) 18 <0> (+6)
inc/class-tool.php 69.69% <90.47%> (+1.29%) 80 <5> (+4) ⬆️

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 04724b4...08f0d0b. Read the comment docs.

@dac514 dac514 merged commit dd72534 into dev May 2, 2019
@dac514 dac514 deleted the prompt-for-auth branch May 2, 2019 18:38
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.

1 participant