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(links): correct website scan API call #358

Merged
merged 1 commit into from
Aug 5, 2020

Conversation

LoneRifle
Copy link
Contributor

@LoneRifle LoneRifle commented Aug 5, 2020

Problem and Solution

The Cloudmersive SDK incorrectly specifies the first argument of
scanWebsite() as a string, the URL itself; its docs clarify that
it should be an explicit request object. Supply the correct value,
and use @ts-ignore to stop TypeScript from complaining

The Cloudmersive SDK incorrectly specifies the first argument of
`scanWebsite()` as a string, the URL itself; its docs clarify that
it should be an explicit request object. Supply the correct value,
and use `@ts-ignore` to stop TypeScript from complaining
@LoneRifle LoneRifle requested a review from yong-jie August 5, 2020 04:51
@LoneRifle LoneRifle merged commit 7f880f1 into develop Aug 5, 2020
@LoneRifle LoneRifle deleted the fix/links/website-scan-api branch August 5, 2020 05:03
xming13 pushed a commit to xming13/GoGovSG that referenced this pull request Aug 21, 2020
The Cloudmersive SDK incorrectly specifies the first argument of
`scanWebsite()` as a string, the URL itself; its docs clarify that
it should be an explicit request object. Supply the correct value,
and use `@ts-ignore` to stop TypeScript from complaining
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