-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Added documentation for DbalSessionHandler as there is none #7132
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
Added documentation for DbalSessionHandler as there is none #7132
Conversation
changelog.rst
Outdated
@@ -13,6 +13,17 @@ documentation. | |||
Do you also want to participate in the Symfony Documentation? Take a look | |||
at the ":doc:`/contributing/documentation/overview`" article. | |||
|
|||
November, 2016 |
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.
The changelogs are automatically generated based on the merge commits. Please remove the changes in this file.
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.
My apologies. I'll remove the change log changes and rebase the latest changes with master.
DbalSessionHandler is probably going to be deprecated. In that case, should we document it? |
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.
This PR seems nice and ready to me (apart a minor comment.
So I'm in favor of merging it. Thanks!
@@ -13,78 +13,6 @@ documentation. | |||
Do you also want to participate in the Symfony Documentation? Take a look | |||
at the ":doc:`/contributing/documentation/overview`" article. | |||
|
|||
October, 2016 |
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.
Those changes should be reverted.
@HeahDude If this documentation is still wanted (seeing as DbalSessionHandler is to be deprecated), I'll revert the |
Closing this issue as the code is being deprecated. |
I've written this documentation for the DbalSessionHandler as there was none, at least, none that was specifically about this class (even though one exists for the PdoSessionHandler).