Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[WebProfilerBundle] Readd Symfony version status in the toolbar#21584

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

@wouterj
Copy link
Member

@wouterjwouterj commentedFeb 11, 2017
edited
Loading

QA
Branch?2.8
Bug fix?yes
New feature?yes
BC breaks?no
Deprecations?no
Tests pass?yes
Fixed tickets-
LicenseMIT
Doc PR-

The Symfony version status ("This Symfony version will no longer receive security fixes." and the like) was no longer shown in the toolbar. This was removed in9d89841 , but the commit description isn't that describing. I guess it's not done on purpose.

I think having this information is crucial for this feature to work: A yellow version doesn't mean anything specific, unless I see "This Symfony will only receive security fixes".

/cc@javiereguiluz

@wouterjwouterj changed the titleReadd Symfony version status in the toolbar[WebProfilerBundle] Readd Symfony version status in the toolbarFeb 11, 2017
@nicolas-grekasnicolas-grekas added this to the2.8 milestoneFeb 11, 2017
Copy link
Member

@javiereguiluzjaviereguiluz left a comment

Choose a reason for hiding this comment

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

👍

@wouterj I'm sorry about this ... but as you said, it wasn't done on purpose. I wanted to remove the<a> tosymfony.com because it's frustrating to click on that element and exit your application and go to symfony.com ... but the description about the version is definitely useful!

@nicolas-grekas
Copy link
Member

Thank you@wouterj.

wouterj reacted with hooray emoji

@nicolas-grekasnicolas-grekas merged commitd714c7e intosymfony:2.8Feb 11, 2017
nicolas-grekas added a commit that referenced this pull requestFeb 11, 2017
…olbar (wouterj)This PR was merged into the 2.8 branch.Discussion----------[WebProfilerBundle] Readd Symfony version status in the toolbar| Q             | A| ------------- | ---| Branch?       | 2.8| Bug fix?      | yes| New feature?  | yes| BC breaks?    | no| Deprecations? | no| Tests pass?   | yes| Fixed tickets | -| License       | MIT| Doc PR        | -The Symfony version status (*"This Symfony version will no longer receive security fixes."* and the like) was no longer shown in the toolbar. This was removed in9d89841 , but the commit description isn't that describing. I guess it's not done on purpose.I think having this information is crucial for this feature to work: A yellow version doesn't mean anything specific, unless I see "This Symfony will only receive security fixes"./cc@javiereguiluzCommits-------d714c7e Readd Symfony version status in the toolbar
@wouterjwouterj deleted the readd-sf-version-status-description branchFebruary 11, 2017 12:00
@fabpotfabpot mentioned this pull requestFeb 17, 2017
@fabpotfabpot mentioned this pull requestMar 6, 2017
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@javiereguiluzjaviereguiluzjaviereguiluz approved these changes

Assignees

No one assigned

Projects

None yet

Milestone

2.8

Development

Successfully merging this pull request may close these issues.

4 participants

@wouterj@nicolas-grekas@javiereguiluz@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp