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

Changing collection_date to collection_timestamp, because time matters #78

Merged
merged 1 commit into from
Jun 11, 2014

Conversation

josenavas
Copy link
Contributor

Just changing the type of collection_date to timestamp (and adjusting the name). Because the time a sample was taken is important.

@squirrelo
Copy link
Contributor

👍 if it doesn't fail

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 89d586e on josenavas:collection_date into ca276b7 on biocore:master.

</path>
<text x='1528' y='345' transform='rotate(90 1528,345)' title='Foreign Key fk_required_sample_info
required_sample_info references required_sample_info_status ( required_sample_info_status_id )' style='fill:#a1a0a0;'>required_sample_info_status_id</text><path transform='translate(7,0)' marker-start='url(#foot)' marker-end='url(#arrow)' d='M 690 780 L 690,855' >
qiita_user references user_level ( user_level_id )' style='fill:#a1a0a0;'>user_level_id</text><path transform='translate(7,0)' marker-start='url(#foot)' marker-end='url(#arrow)' d='M 690 780 L 690,855' >
Copy link
Member

Choose a reason for hiding this comment

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

Any reason for having extra white spaces?

antgonza added a commit that referenced this pull request Jun 11, 2014
Changing collection_date to collection_timestamp, because time matters
@antgonza antgonza merged commit 29c797e into qiita-spots:master Jun 11, 2014
@josenavas josenavas deleted the collection_date branch June 11, 2014 22:20
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.

None yet

4 participants