Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Form] Fix #10658 Deprecate "read_only" option #11376

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 2 commits into from
Closed

[Form] Fix #10658 Deprecate "read_only" option #11376

wants to merge 2 commits into from

Conversation

rpg600
Copy link
Contributor

@rpg600 rpg600 commented Jul 11, 2014

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? yes
Tests pass? yes
Fixed tickets #10658
License MIT
Doc PR symfony/symfony-docs#3782

@ClementGautier
Copy link
Contributor

@rpg600 Can you squash @snoob second commit with the first one plz ?

@snoob
Copy link
Contributor

snoob commented Jul 16, 2014

I have done your changes on the root PR (#10676). It's useless to create a new one for 3 lines it's avoid duplication and simplify the job for the mergers. If something is still missing just report it on my PR.

@Tobion
Copy link
Contributor

Tobion commented Jul 16, 2014

Since I found some more changes to be done, let's just continue with the original PR that is still active.

@Tobion Tobion closed this Jul 16, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants