Skip to content

Conversation

mikeengland
Copy link

@mikeengland mikeengland commented Dec 20, 2016

@StyXman I created a new PR. Let me know if you need anything changed.

Old PR was #117

@mohamedrez
Copy link

mohamedrez commented Jan 12, 2017

@manosim
Copy link
Owner

manosim commented Jan 12, 2017

Looks like @mohamedrez is correct - this was fixed in #127 . Should we close this one @mikeengland?

@mikeengland
Copy link
Author

mikeengland commented Jan 12, 2017

Feel free, although a comment here said that some changes from this PR would still be needed #127 (117 was the old version of this PR)

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.

4 participants