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

Editing a section with the last visit field causes the last visit field to add itselfs again. #4

Closed
wdebusschere opened this Issue Aug 30, 2017 · 5 comments

Comments

Projects
None yet
2 participants
@wdebusschere

wdebusschere commented Aug 30, 2017

Symphony 2.7.0

@wdebusschere wdebusschere changed the title from Editing a section with the last visit field causes the last visit field to add itselfs again to Editing a section with the last visit field causes the last visit field to add itselfs again. Aug 30, 2017

@twiro twiro self-assigned this Nov 30, 2017

@twiro twiro added the bug label Nov 30, 2017

@twiro twiro added this to the 1.1.0 milestone Nov 30, 2017

@twiro

This comment has been minimized.

Show comment
Hide comment
@twiro

twiro Nov 30, 2017

Owner

Here's why: symphonycms/symphony-2#2736

Will fix this in the near future!

Owner

twiro commented Nov 30, 2017

Here's why: symphonycms/symphony-2#2736

Will fix this in the near future!

twiro added a commit that referenced this issue Dec 4, 2017

Release 2.0.0
- Code clean up
- Styling improvements: uses the 'field-value-readonly'-class now
(Symphony 2.6.5 +)
- Compatibility with Symphony 2.7.x
([#4](#4)
- PHP 7 support
@twiro

This comment has been minimized.

Show comment
Hide comment
@twiro

twiro Dec 4, 2017

Owner

Hey Wannes – could you give the current integration-branch a try?

Owner

twiro commented Dec 4, 2017

Hey Wannes – could you give the current integration-branch a try?

@wdebusschere

This comment has been minimized.

Show comment
Hide comment
@wdebusschere

wdebusschere Dec 4, 2017

Problem solved! Thanks.

wdebusschere commented Dec 4, 2017

Problem solved! Thanks.

@twiro

This comment has been minimized.

Show comment
Hide comment
@twiro

twiro Dec 4, 2017

Owner

Cool! And do you still get any error regarding processRawFieldData()?
If not I would officially release the integration branch as 2.0.0 as I didn't run in any more errors in my tests.

Owner

twiro commented Dec 4, 2017

Cool! And do you still get any error regarding processRawFieldData()?
If not I would officially release the integration branch as 2.0.0 as I didn't run in any more errors in my tests.

@wdebusschere

This comment has been minimized.

Show comment
Hide comment
@wdebusschere

wdebusschere Dec 4, 2017

No more errors now. Thank you

wdebusschere commented Dec 4, 2017

No more errors now. Thank you

@twiro twiro referenced this issue Dec 5, 2017

Merged

Release 2.0.0 #5

@twiro twiro closed this Dec 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment