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

TopSites #7

Closed
danieler1981 opened this issue Dec 3, 2016 · 3 comments
Closed

TopSites #7

danieler1981 opened this issue Dec 3, 2016 · 3 comments
Assignees

Comments

@danieler1981
Copy link

Why TopSites is not supported?

@lupomontero lupomontero self-assigned this Dec 5, 2016
@lupomontero
Copy link
Member

Hi @danieler1981,

Thanks for the suggestion. I hadn't added "support" for TopSites as this is different service/API. See:

However, both services seem to share the same query API, so I have added "support" for TopSites to this module, and so far it seems to work ;-) You can find an example here:

https://github.com/wrangr/awis/blob/master/test/awis.spec.js#L257

Best,

L

@domahub
Copy link

domahub commented Mar 25, 2017

Can you add this to the readme? Real great stuff, thanks 👍

@lupomontero
Copy link
Member

@domahub: thanks for the positive feedback ;-)

I've just added a TopSites example to the readme: 50a43f5

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

No branches or pull requests

3 participants