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

Support Symfony 3 #36

Merged
merged 13 commits into from
Apr 29, 2017
Merged

Support Symfony 3 #36

merged 13 commits into from
Apr 29, 2017

Conversation

gonzaloalonsod
Copy link
Contributor

No description provided.

SCDatetimeType::class symfony 3
@koprivajakub
Copy link

Seems fine to me, I would need the Symfony 3 support for my project. So it would be nice to merge it ASAP 👍
cc: @stephanecollot

@gonzaloalonsod
Copy link
Contributor Author

Lo implemente aqui: https://github.com/MWSimple/AdminCrudBundle
Espero puedan aceptar los cambios si estan bien.
Saludos!

*
*/
class DatetimeType extends AbstractType
class SCDatetimeType extends AbstractType
Copy link

Choose a reason for hiding this comment

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

What is the reason for renaming the class as this is a BC break?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Porque daba un error de que DatetimeType ya existe.
Because it gives an error that Datetime Type already exists.

Copy link

Choose a reason for hiding this comment

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

Can you please show the full error message and its stack trace?

@@ -1,6 +1,6 @@
{
"name": "stephanecollot/datetimepicker-bundle",
"version": "1.0.1",
"version": "2.0.0",
Copy link

Choose a reason for hiding this comment

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

should be reverted too

@@ -1,7 +1,7 @@
<?php

/*
* This file is part of the SCDatetimepickerBundle package.
* This file is part of the DatetimepickerBundle package.
Copy link

Choose a reason for hiding this comment

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

looks wrong to me

@gonzaloalonsod
Copy link
Contributor Author

@koprivajakub
Seems fine to me, I would need the Symfony 3 support for my project. So it would be nice to merge it ASAP 👍
cc: @stephanecollot

@johnnyfekete
Copy link

@stephanecollot could you merge the changes these guys made? :) I'd be needed to upgrade to Symfony 3 :)

@gonzaloalonsod
Copy link
Contributor Author

If testing use "mwsimple/datetimepicker-bundle": "2.0.0",

@gonzaloalonsod
Copy link
Contributor Author

SF3 https://github.com/MWSimple/DatetimepickerBundle/tree/2.0

@stephanecollot stephanecollot merged commit ebecd02 into stephanecollot:master Apr 29, 2017
@gonzaloalonsod
Copy link
Contributor Author

@gonzaloalonsod
Copy link
Contributor Author

@c960657
Copy link

c960657 commented May 12, 2021

For Symphony 3.4, this is also needed: #45

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.

6 participants