Skip to content

Commit d24fe3e

Browse files
committed
Merge pull request #8 from danielfarrell/fix-example-config
Strategy should not be in a list
2 parents fd8e660 + 352255b commit d24fe3e

2 files changed

Lines changed: 6 additions & 6 deletions

File tree

README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,14 +27,14 @@
2727
```elixir
2828
config :ueberauth, Ueberauth,
2929
providers: [
30-
github: [{Ueberauth.Strategy.GitHub, []}]
30+
github: {Ueberauth.Strategy.Github, []}
3131
]
3232
```
3333

3434
1. Update your provider configuration:
3535

3636
```elixir
37-
config :ueberauth, Ueberauth.Strategy.GitHub.OAuth,
37+
config :ueberauth, Ueberauth.Strategy.Github.OAuth,
3838
client_id: System.get_env("GITHUB_CLIENT_ID"),
3939
client_secret: System.get_env("GITHUB_CLIENT_SECRET")
4040
```
@@ -79,7 +79,7 @@ By default the requested scope is "user,public\_repo". Scope can be configured e
7979
```elixir
8080
config :ueberauth, Ueberauth,
8181
providers: [
82-
github: {Ueberauth.Strategy.GitHub, [default_scope: "user,public_repo,notifications"]}
82+
github: {Ueberauth.Strategy.Github, [default_scope: "user,public_repo,notifications"]}
8383
]
8484
```
8585

lib/ueberauth/strategy/github.ex

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ defmodule Ueberauth.Strategy.Github do
1212
1313
config :ueberauth, Ueberauth,
1414
providers: [
15-
github: [ { Ueberauth.Strategy.Github, [] } ]
15+
github: { Ueberauth.Strategy.Github, [] }
1616
]
1717
1818
Then include the configuration for github.
@@ -54,7 +54,7 @@ defmodule Ueberauth.Strategy.Github do
5454
5555
config :ueberauth, Ueberauth,
5656
providers: [
57-
github: [ { Ueberauth.Strategy.Github, [uid_field: :email] } ]
57+
github: { Ueberauth.Strategy.Github, [uid_field: :email] }
5858
]
5959
6060
Default is `:login`
@@ -63,7 +63,7 @@ defmodule Ueberauth.Strategy.Github do
6363
6464
config :ueberauth, Ueberauth,
6565
providers: [
66-
github: [ { Ueberauth.Strategy.Github, [default_scope: "user,public_repo"] } ]
66+
github: { Ueberauth.Strategy.Github, [default_scope: "user,public_repo"] }
6767
]
6868
6969
Deafult is "user,public_repo"

0 commit comments

Comments
 (0)