Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

Updated field names in course summaries. #146

Merged
merged 1 commit into from
Dec 5, 2016
Merged

Conversation

dsjen
Copy link
Contributor

@dsjen dsjen commented Dec 5, 2016

  • renamed start/end_time to start/end_date
  • renamed modes to enrollment_modes

I misnamed the columns that was generated by the data pipeline. This PR fixes that.

* renamed start/end_time to start/end_date
* renamed modes to enrollment_modes
@coveralls
Copy link

Coverage Status

Coverage remained the same at 98.194% when pulling 915cab3 on dsjen/course-list-fields into e122056 on master.

Copy link
Contributor

@thallada thallada left a comment

Choose a reason for hiding this comment

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

👍

@dsjen dsjen merged commit 72c71de into master Dec 5, 2016
@dsjen dsjen deleted the dsjen/course-list-fields branch December 5, 2016 21:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants