Skip to content

Conversation

@peterrehm
Copy link
Contributor

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

Relates to #14825 and #14377. The behaviour was changed with #16681 so a not in the upgrade information makes sense to me.

UPGRADE-3.0.md Outdated
Copy link
Member

Choose a reason for hiding this comment

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

and setting it to something different is deprecated

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated accordingly.

Copy link
Member

Choose a reason for hiding this comment

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

will default -> defaults

@peterrehm
Copy link
Contributor Author

@xabbuh Is this good to go?

@xabbuh
Copy link
Member

xabbuh commented Dec 17, 2015

👍 LGTM

UPGRADE-3.0.md Outdated
Copy link
Member

Choose a reason for hiding this comment

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

'choice' -> ChoiceType::class

@peterrehm
Copy link
Contributor Author

@nicolas-grekas Good catch, updated.

@nicolas-grekas
Copy link
Member

Thank you @peterrehm.

@nicolas-grekas nicolas-grekas merged commit 28675c9 into symfony:2.8 Dec 18, 2015
nicolas-grekas added a commit that referenced this pull request Dec 18, 2015
This PR was merged into the 2.8 branch.

Discussion
----------

Upgrade information for the choice_value option

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

Relates to #14825 and #14377. The behaviour was changed with #16681 so a not in the upgrade information makes sense to me.

Commits
-------

28675c9 Reflected the change of the choice_value option in the Upgrade information
@peterrehm peterrehm deleted the choice-value branch December 18, 2015 11:46
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