Closed mccutchen closed 5 years ago
Thanks @mccutchen -- couple small comments, but this looks great!
Merging #263 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #263 +/- ##
=======================================
Coverage 62.14% 62.14%
=======================================
Files 53 53
Lines 4169 4169
=======================================
Hits 2591 2591
Misses 1391 1391
Partials 187 187
Problem
In the course of working on https://github.com/buzzfeed/sso/pull/262, I found a couple of small issues with the quickstart guide: the quickstart-specific suggested OAuth callback URLs in the google & okta provider setup docs were incomplete.
Solution
FIrst, fix the suggested URLs. I also made a few more invasive/opinionated wording/structure changes to the quickstart guide itself, if these changes do not seem like an improvement to y'all I will revert them!