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

Fix #16 #17

Merged
merged 1 commit into from
Jan 16, 2015
Merged

Fix #16 #17

merged 1 commit into from
Jan 16, 2015

Conversation

markitosgv
Copy link
Contributor

No description provided.

@javiereguiluz
Copy link
Collaborator

Very nice catch! I totally missed it. Thanks @markitosgv

@javiereguiluz javiereguiluz merged commit 269f460 into EasyCorp:master Jan 16, 2015
javiereguiluz added a commit that referenced this pull request Jan 16, 2015
This PR was merged into the master branch.

Discussion
----------

Fix #16

Commits
-------

269f460 Fix #16
@xabbuh
Copy link
Collaborator

xabbuh commented Jan 16, 2015

@javiereguiluz Does it make sense to maintain a copy of the form theme from the Symfony core here?

@javiereguiluz
Copy link
Collaborator

@xabbuh unfortunately yes, because I want this bundle to work flawless for Symfony 2.3+ and that version doesn't include the cool Bootstrap form themes (they were introduced in Symfony 2.6).

@xabbuh
Copy link
Collaborator

xabbuh commented Jan 17, 2015

@javiereguiluz Indeed, I didn't think about the fact that it was actually only included recently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants