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.

Sign up for GitHub

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

Jump to bottom

Algolia indexing needs tweaked on CDNJS.com #4513

Closed
9mm opened this issue Apr 2, 2015 · 4 comments
Closed

Algolia indexing needs tweaked on CDNJS.com #4513

9mm opened this issue Apr 2, 2015 · 4 comments
Labels
💡 Help wanted

Comments

@9mm
Copy link

9mm commented Apr 2, 2015

I just implemented Algolia on my own site, and I love it!

That's unrelated to the problem I saw, however, which is that you have some things coming up as undefined.

cdnjs_com_-_the_missing_cdn_for_javascript_and_css
PeterDaveHello added a commit that referenced this issue Apr 2, 2015
@PeterDaveHello
change field repository to repositories in backbone.radio, cc #4513
5aa38e1
@PeterDaveHello
Copy link
Contributor

Thanks, let's see if this will work : 5aa38e1, if so, I think we should make it support repository field.

@PeterDaveHello
Copy link
Contributor

Hmmmm ... not work at all ...

@PeterDaveHello
Copy link
Contributor

@thomasdavis : Would you please also take a look at this issue? thanks!

@PeterDaveHello
Copy link
Contributor

@9mm The problem came from the moved repo, and the node-github api we used can't handle this kind of case, I opened issue here octokit/octokit.js#257, and updated the repo url in f5603d8, so close this issue, thanks for your report.

PeterDaveHello added a commit to cdnjs/new-website that referenced this issue May 30, 2015
@PeterDaveHello
add warning if got empty data when enriching libs
242c632 
cc: cdnjs/cdnjs#4513
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
💡 Help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@9mm @PeterDaveHello