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

Time issue fix #46

Merged
merged 4 commits into from
Sep 17, 2018
Merged

Time issue fix #46

merged 4 commits into from
Sep 17, 2018

Conversation

gwenchee
Copy link
Collaborator

This PR addresses #44.

@gwenchee gwenchee added the bug label Sep 17, 2018
@gwenchee gwenchee self-assigned this Sep 17, 2018
@jbae11
Copy link
Collaborator

jbae11 commented Sep 17, 2018

Looks good to me!
Also the addition in line 235 is to ensure that only Facility types were added to commod_to_fac. For some reason it was adding insitutions.

@jbae11 jbae11 merged commit da607fc into arfc:correct-develop Sep 17, 2018
gwenchee referenced this pull request in gwenchee/d3ploy Sep 17, 2018
Merge pull request #46 from gwenchee/time-issue-fix
@gwenchee gwenchee mentioned this pull request Oct 1, 2018
FlanFlanagan pushed a commit that referenced this pull request Nov 9, 2018
gwenchee pushed a commit that referenced this pull request Mar 11, 2019
gwenchee added a commit that referenced this pull request Mar 11, 2019
Merge pull request #46 from arfc/develop
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