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

updates Node APIs #53

Merged
merged 1 commit into from Oct 2, 2019
Merged

updates Node APIs #53

merged 1 commit into from Oct 2, 2019

Conversation

@AndriusA
Copy link
Collaborator

AndriusA commented Oct 2, 2019

  • extra result field in case error happens in processing request
  • new AdBlockClient.Engine accepts JSON object for second argument specifying debug and optimize parameters
  • utility method checking if provided 3 arguments (url, source_url, type) form a valid request
- extra result field in case error happens in processing request
- `new AdBlockClient.Engine` accepts JSON object for second argument specifying `debug` and `optimize` parameters
- utility method checking if provided 3 arguments (url, source_url, type) form a valid request
@AndriusA AndriusA requested a review from pes10k Oct 2, 2019
@pes10k
pes10k approved these changes Oct 2, 2019
@AndriusA AndriusA merged commit babd5a3 into master Oct 2, 2019
2 of 4 checks passed
2 of 4 checks passed
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
Travis CI - Branch Build Passed
Details
Travis CI - Pull Request Build Passed
Details
@AndriusA AndriusA deleted the node-apis branch Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.