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

Assign attributes directly during import validation #141

Closed
wants to merge 3 commits into from
Closed

Assign attributes directly during import validation #141

wants to merge 3 commits into from

Conversation

trandoanhung1991
Copy link
Contributor

To prevent parsing UTC time into local datetime

* Thibaud Guillaume-Gentil
* Mark Van Holstyn
* Victor Costan
Tioeuaeuesting webhook to Jenkins
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't think you meant to commit this?

@jkowens
Copy link
Collaborator

jkowens commented Feb 21, 2016

@trandoanhung1991 If you remove your change to README.markdown and squash this into a single commit that would probably help move this along. I think there's been another issue discovered that this fix could resolve. See #235.

I would also suggest referencing the related issue #140 that you opened in the description. 😄

@trandoanhung1991
Copy link
Contributor Author

For some reason I can't find this repo anywhere. I'll close this PR and create a new one, is that ok?

@trandoanhung1991
Copy link
Contributor Author

Just in case, the new PR is #237

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