Skip to content

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

Closed
wants to merge 3 commits into from
Closed

Added documentation for DbalSessionHandler as there is none #7132

wants to merge 3 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Nov 10, 2016

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).

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
Copy link
Member

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.

Copy link
Author

@ghost ghost Nov 11, 2016

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.

@xabbuh xabbuh added On hold and removed On hold labels Nov 12, 2016
@javiereguiluz
Copy link
Member

DbalSessionHandler is probably going to be deprecated. In that case, should we document it?

xabbuh added a commit that referenced this pull request Nov 22, 2016
…the changelog (wouterj)

This PR was merged into the 2.7 branch.

Discussion
----------

Added a comment to prevent people from manually updating the changelog

For instance, to prevent things like #7132

Commits
-------

6958b7c Added a comment to prevent people from manually updating the changelog
Copy link
Contributor

@HeahDude HeahDude left a 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
Copy link
Contributor

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 HeahDude added this to the 2.7 milestone Jul 29, 2017
@HeahDude HeahDude changed the base branch from master to 2.7 July 29, 2017 12:08
@HeahDude HeahDude changed the base branch from 2.7 to master July 29, 2017 12:09
@ghost
Copy link
Author

ghost commented Aug 7, 2017

@HeahDude If this documentation is still wanted (seeing as DbalSessionHandler is to be deprecated), I'll revert the changelog.rst change tonight.

@fabpot
Copy link
Member

fabpot commented Oct 5, 2017

Closing this issue as the code is being deprecated.

@fabpot fabpot closed this Oct 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants