Skip to content
This repository has been archived by the owner on Nov 21, 2018. It is now read-only.

/sh_main.js and /sh_javascript.min.js result in 404s: #51

Closed
alrra opened this issue Jan 14, 2015 · 6 comments
Closed

/sh_main.js and /sh_javascript.min.js result in 404s: #51

alrra opened this issue Jan 14, 2015 · 6 comments
Labels

Comments

@alrra
Copy link

alrra commented Jan 14, 2015

The /sh_main.js and /sh_javascript.min.js scripts result in 404s:

$ curl -sSLI https://iojs.org/sh_javascript.min.js

HTTP/1.1 404 Not Found
Server: nginx/1.4.6 (Ubuntu)
Date: Wed, 14 Jan 2015 10:23:08 GMT
Content-Type: text/html
Content-Length: 177
Connection: keep-alive

$ curl -sSLI https://iojs.org/sh_main.js

HTTP/1.1 404 Not Found
Server: nginx/1.4.6 (Ubuntu)
Date: Wed, 14 Jan 2015 10:24:22 GMT
Content-Type: text/html
Content-Length: 177
Connection: keep-alive
@Fishrock123
Copy link
Contributor

@chrisdickinson

@billautomata
Copy link

the font doesn't load either

screen shot 2015-01-14 at 8 15 12 am

@iamphill
Copy link

Just done a fork of the main repo. Going to fix issues now

@chrisdickinson
Copy link

These files should be made available on the website, ideally. They handle syntax highlighting which will be useful for blog posts, and the docs were depending on them. It's a bit of a regression on the website side.

The changes for the font have been merged last night.

This points at another issue: should the docs team (responsible primarily for doc generation) be separate from the website team? I was hoping to hand the docs team full control over the /docs/ path of the website.

@chrisdickinson
Copy link

(NB: while its a regression, it's kind of an easy one to miss with the current dev setup -- one has to glom the website together with the generated docs to find out the files are missing.)

chrisdickinson pushed a commit to nodejs/node that referenced this issue Jan 15, 2015
Updated Makefile to remove special casing for those files, and
moved the files to doc/api_assets.

Fixes: nodejs/iojs.org#51
PR-URL: #409
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
@chrisdickinson
Copy link

This should be fixed as of nodejs/node@e789103.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

6 participants