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.

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

DOCS Authentication #7102

Merged
merged 1 commit into from
Sep 6, 2017
Merged

Conversation

Firesphere
Copy link
Contributor

Replaces #7008

@Firesphere Firesphere changed the title Authentication documentation rewrite DOCS Authentication Jul 3, 2017
extensions:
- MyMemberExtension
```yml
Member:
Copy link
Contributor

Choose a reason for hiding this comment

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

Existing issue, but this should be SilverStripe\Security\Member

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch, should've been updated by now :) (multiple places)

@Firesphere Firesphere force-pushed the pulls/auth-docs branch 2 times, most recently from ca120a3 to 6b72954 Compare July 3, 2017 08:45
@flamerohr flamerohr changed the base branch from master to 4 July 5, 2017 11:47
@Firesphere
Copy link
Contributor Author

Please rebuild. I can't imagine some .md files breaking the build 😂

@dhensby dhensby force-pushed the pulls/auth-docs branch 2 times, most recently from 60ea86c to 3669af9 Compare July 19, 2017 17:34
@@ -1391,7 +1392,7 @@ specific functions.

#### Upgrading custom Authenticators

The methods `register` and `unregister` on `Authenticator` are deprecated in favor of the `Config` system. This means that any custom Authenticator needs to be registered through the yml config:
The methods `register` and `unregister` on `Authenticator` are deprecated in favor of the `Config` system. This means that any custom Authenticator needs to be registered through the yml config. For further information on how to create a custom authentication method, [see the Authention documentation](/developer_guides/Security/Authentication).
Copy link
Contributor

Choose a reason for hiding this comment

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

typo in "authentication"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@tractorcow tractorcow merged commit b270eee into silverstripe:4 Sep 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants