Skip to content

Conversation

@singingwolfboy
Copy link
Contributor

STUD-1072

Copy link
Contributor

Choose a reason for hiding this comment

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

Perhaps a comment here as to why you needed to override the default and what the assumptions are on the override (i.e., that you needed to make it in the past)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added a comment. @dmitchell, can you verify that it makes sense and is accurate?

@dmitchell
Copy link
Contributor

Also, update changelog.rst. Otherwise, 👍

@andy-armstrong
Copy link
Contributor

👍

@singingwolfboy
Copy link
Contributor Author

I updated the changelog.

singingwolfboy added a commit that referenced this pull request Jan 7, 2014
Set default section start date to Jan 1, 2030
@singingwolfboy singingwolfboy merged commit 5d63cd0 into master Jan 7, 2014
@singingwolfboy singingwolfboy deleted the db/change-section-start-date branch January 7, 2014 21:10
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Aug 24, 2017
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.

4 participants