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.

Sign up for GitHub

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

Jump to bottom

Feature/allow specifing users withaccess #5

Merged
merged 12 commits into from
Jul 18, 2017
Merged

Feature/allow specifing users withaccess #5

merged 12 commits into from
Jul 18, 2017

Conversation

jimmypuckett
Copy link
Member

No description provided.

ssfinney
ssfinney reviewed Jul 18, 2017
View reviewed changes
tests/Controllers/SsoControllerTest.php Outdated
*/
public function it_uses_the_configured_secret_with_the_helper()
public function it_aborts_if_the_paylod_is_invalid()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"paylod" -> "payload"

@coveralls
Copy link

coveralls commented Jul 18, 2017 •
edited
Loading

Coverage Status

Coverage increased (+1.7%) to 84.091% when pulling 307b856 on feature/allowSpecifingUsersWithaccess into 80a0047 on develop.

@coveralls
Copy link

coveralls commented Jul 18, 2017 •
edited
Loading

Coverage Status

Coverage increased (+1.7%) to 84.091% when pulling d35777b on feature/allowSpecifingUsersWithaccess into 80a0047 on develop.

@ssfinney ssfinney merged commit 042002e into develop Jul 18, 2017
@ssfinney ssfinney deleted the feature/allowSpecifingUsersWithaccess branch July 18, 2017 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ssfinney ssfinney ssfinney left review comments

Assignees

@ssfinney ssfinney

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jimmypuckett @coveralls @ssfinney