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

NEW: Added details to README regarding EditableFormField column clean ta... #144

Merged
merged 1 commit into from
Aug 9, 2013

Conversation

kmayo-ss
Copy link
Contributor

@kmayo-ss kmayo-ss commented Aug 8, 2013

Added tasks section to README advising of the EditableFromField column clean task

@wilr
Copy link
Member

wilr commented Aug 9, 2013

Run into the issue more? Perhaps we make the task auto run on requireDefaultRecords to clean up after dev/build (or is it dev/build triggering the error though?). Also NEW is for new features/api's and is highlighted in the changelog. This is minor so doesn't need a prefix.

@kmayo-ss
Copy link
Contributor Author

kmayo-ss commented Aug 9, 2013

Have not run into the issue anymore have only seen the issue once which was #119 which was when upgrading the module.
I believe it could be used after a dev/build (would need to confirm with testing) but as I have only seen it occur once it does not seem to be that important.

wilr added a commit that referenced this pull request Aug 9, 2013
Added details to README regarding EditableFormField column clean ta...
@wilr wilr merged commit af69b0d into silverstripe:master Aug 9, 2013
@wilr
Copy link
Member

wilr commented Aug 9, 2013

Sweet just checking this wasn't a large blocker to several site updates.

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.

2 participants