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 upstream #447 #451

Closed

Conversation

matt-alvey-eventweaver
Copy link

These changes should fix the problem described in #447

These changes should fix the problem described in yql#447
@johnmwilson82
Copy link

This didn't work when I tested it, I've added a pull request of my own with a couple of extra changes in the section where it pulls out the text for param_name and value.

@matt-alvey-eventweaver
Copy link
Author

Closing in favor of the updated pull request from @johnmwilson82.

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.

3 participants