-
Notifications
You must be signed in to change notification settings - Fork 821
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
[WIP] Security docs and changelog updates for auth refactor. #7008
Conversation
b02fee3
to
bc42e5c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Firesphere Can you please rebase this in the next 24h, and trim down the amount of upgrading guidance? We need to get the upgrading docs in there for the beta1 release, which is imminent.
SilverStripe\Security\Authenticator; | ||
authenticators: | ||
- MyVendor\MyModule\MyAuthenticator | ||
SilverStripe\Core\Injector\Injector: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most of this should move to the actual security docs - it's less of an upgrade concern. The changelog is getting far too long with this stuff all inlined.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, will do asap :)
@Firesphere, a rebate would be better than adding a merge commit and the commits need squashing as the commit messages are not quite optimal at the moment ;) |
No description provided.