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

Set error messages to link to relevant docs #17

Open
pop opened this issue Aug 7, 2015 · 3 comments
Open

Set error messages to link to relevant docs #17

pop opened this issue Aug 7, 2015 · 3 comments

Comments

@pop
Copy link
Contributor

pop commented Aug 7, 2015

Error messages should link to appropriate documentation in addition to an error message.

@osuosl/devs @jordane @Kennric thoughts?

@pop pop added the enhancement label Aug 7, 2015
@jordane
Copy link

jordane commented Aug 10, 2015

to summarize the IRC conversation:

  • giving a direct url means we commit to maintaing those docs for a very long time
  • giving a direct url mayhave issues if someone else modifies code but doesn't change the documentation links
  • giving some other reference to the docs (error code, chapter & section) could be useful, but also has disadvantages
  • this is ultimately a really good idea, but we should be careful about how we do it.

@iankronquist
Copy link
Contributor

Why not make a /docs endpoint which hosts the docs which are already in the repo, and then you can provide links to that? If they live on the same server they shouldn't be out of date or unreachable.

  • The docs should be maintained as long as the app is around.
  • You can get the url dynamically like "http://#{fqdn}/docs/error_4.html" or something.
  • I think we have this already?

@iankronquist
Copy link
Contributor

https://github.com/osuosl/timesync-node/compare/docs-spike
Obviously you have to build the docs.

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

No branches or pull requests

4 participants