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

Fix btax backend bug caused by PR #641 #651

Merged
merged 5 commits into from Sep 18, 2017

Conversation

hdoupe
Copy link
Collaborator

@hdoupe hdoupe commented Sep 18, 2017

Address #637 and #641

@brittainhard
Copy link
Contributor

@hdoupe have you tested this fix yet?

@hdoupe
Copy link
Collaborator Author

hdoupe commented Sep 18, 2017

@brittainhard Yes, it seemed to work fine.

@hdoupe
Copy link
Collaborator Author

hdoupe commented Sep 18, 2017

@brittainhard is everything working OK?

@brittainhard
Copy link
Contributor

@hdoupe testing it right now.

@hdoupe
Copy link
Collaborator Author

hdoupe commented Sep 18, 2017

Great thanks @brittainhard

@brittainhard
Copy link
Contributor

@hdoupe looks like its fixed! I'll get this PR merged.

@hdoupe
Copy link
Collaborator Author

hdoupe commented Sep 18, 2017

Awesome, thanks for taking a look at it.

@brittainhard brittainhard merged commit cf0d3e7 into ospc-org:634_btax_dataframes Sep 18, 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.

None yet

2 participants