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

[WebProfilerBundle] Removed the deprecated web_profiler.position option #23728

Merged
merged 1 commit into from
Sep 9, 2017

Conversation

javiereguiluz
Copy link
Member

Q A
Branch? master
Bug fix? no
New feature? no
BC breaks? no
Deprecations? yes
Tests pass? yes
Fixed tickets #23528
License MIT
Doc PR -

This PR shows the things we could remove if we accept #23528. If it's accepted, I'd create the PR for 3.4 deprecating the feature instead of removing it.

@xabbuh xabbuh modified the milestones: 3.4, 4.0 Aug 1, 2017
@javiereguiluz javiereguiluz changed the title Deprecated the web_profiler.position option Removed the deprecated web_profiler.position option Sep 3, 2017
fabpot added a commit that referenced this pull request Sep 7, 2017
…iluz)

This PR was squashed before being merged into the 3.4 branch (closes #24080).

Discussion
----------

Deprecated the web_profiler.position option

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | no
| New feature?  | no <!-- don't forget updating src/**/CHANGELOG.md files -->
| BC breaks?    | no
| Deprecations? | yes
| Tests pass?   | yes
| Fixed tickets | #23528
| License       | MIT
| Doc PR        | -

Related to #23728, which removes the feature for Symfony 4.0.

Commits
-------

53387b4 Deprecated the web_profiler.position option
@nicolas-grekas nicolas-grekas changed the title Removed the deprecated web_profiler.position option [WebProfilerBundle] Removed the deprecated web_profiler.position option Sep 9, 2017
@nicolas-grekas
Copy link
Member

Thank you @javiereguiluz.

@nicolas-grekas nicolas-grekas merged commit cafdf2b into symfony:master Sep 9, 2017
nicolas-grekas added a commit that referenced this pull request Sep 9, 2017
…r.position option (javiereguiluz)

This PR was merged into the 4.0-dev branch.

Discussion
----------

[WebProfilerBundle] Removed the deprecated web_profiler.position option

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | yes
| Tests pass?   | yes
| Fixed tickets | #23528
| License       | MIT
| Doc PR        | -

This PR shows the things we could remove if we accept #23528. If it's accepted, I'd create the PR for 3.4 deprecating the feature instead of removing it.

Commits
-------

cafdf2b [WebProfilerBundle] Deprecated the web_profiler.position option
@fabpot fabpot mentioned this pull request Oct 19, 2017
ostrolucky pushed a commit to ostrolucky/symfony that referenced this pull request Mar 25, 2018
…viereguiluz)

This PR was squashed before being merged into the 3.4 branch (closes symfony#24080).

Discussion
----------

Deprecated the web_profiler.position option

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | no
| New feature?  | no <!-- don't forget updating src/**/CHANGELOG.md files -->
| BC breaks?    | no
| Deprecations? | yes
| Tests pass?   | yes
| Fixed tickets | symfony#23528
| License       | MIT
| Doc PR        | -

Related to symfony#23728, which removes the feature for Symfony 4.0.

Commits
-------

53387b4 Deprecated the web_profiler.position option
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.

4 participants