dkoslicki / NCATS

MIT License
2 stars 0 forks source link

http://ncats.saramsey.org/doc/RTX/index.html returns 404 #123

Closed erikyao closed 6 years ago

erikyao commented 6 years ago

RTX OpenAPI page has a link with tag "Documentation for the RTX web service" to http://ncats.saramsey.org/doc/RTX/index.html, which would return 404.

Suggested Fix:

  1. Comment out the corresponding config in code/UI/OpenAPI/python-flask-server/swagger_server/swagger/swagger.yaml
  2. Then restart the swagger server by running python3 -m swagger_server under code/UI/OpenAPI/python-flask-server
## Comment out these 3 lines 

# externalDocs:
#   description: "Documentation for the RTX web service"
#   url: "http://ncats.saramsey.org/doc/RTX/index.html"
edeutsch commented 6 years ago

I am fixing the web service definition at this very moment. It didn't port as well as I have thought from ncats to lysine. I'll post another message once I've fixed lysine. and then if you could test again, that would be appreciated!

saramsey commented 6 years ago

@erikyao can you put the same file into

/var/www/html/doc/RTX

on lysine? (make sure to do it inside the container)

erikyao commented 6 years ago

@saramsey right on it. Thanks! @edeutsch sorry I got it wrong. We are not going to remove that link but create a page for that link.

edeutsch commented 6 years ago

yep, hold on, i'm working on a fix, that is obsolete stuff.

erikyao commented 6 years ago

@edeutsch now there is an index.html on both ncats.saramsey.org and lysine.ncats.io/ dockers, located at

/var/www/html/doc/RTX/index.html

with the symbol link

/var/www/html -> /mnt/data/orangeboard/code/NCATS/code/UI/interactive/

This index.html is a http-equiv="refresh" page directed to our github repo:

<html>
  <meta http-equiv="refresh" content="0,URL=https://github.com/dkoslicki/NCATS" />
  <p>
    If your browser does not automatically redirect, please click on the
    <a href="https://github.com/dkoslicki/NCATS">RTX home page</a>
  </p>
</html>
edeutsch commented 6 years ago

okay, the above URL was an early idea for where it might be and is now gone on lysine (I hope), so this should not be necessary, but fine that it's there.

I still need to replicate to ncats