-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Update GitHub issue templates #7554
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
Conversation
@LiamConnors can you please review wording as well? thx |
.github/ISSUE_TEMPLATE/bug_report.md
Outdated
Thanks for your interest in plotly.js! | ||
|
||
- Before submitting a new bug report, please search for existing and closed issues. If your bug is not addressed yet, fill out the sections below and submit a new issue. | ||
- Issues found on the example pages from https://plot.ly/javascript/ should be filed in our [documentation repo](https://github.com/plotly/documentation/issues) with the exception of https://plot.ly/javascript/reference which should be filed here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Issues found on the example pages from https://plot.ly/javascript/ should be filed in our [documentation repo](https://github.com/plotly/documentation/issues) with the exception of https://plot.ly/javascript/reference which should be filed here. | |
- Issues found on the example pages from https://plot.ly/javascript/ should be filed in our [documentation repo](https://github.com/plotly/documentation/issues) with the exception of https://plotly.com/javascript/reference which should be filed here. |
The docs sites are more commonly accessed at the plotly.com domain
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated for all the links. Thanks for your eagle eyes!
### Steps to reproduce | ||
|
||
_Reports **must** include steps to reproduce the issue. Alternatively, you can use [codepen](http://codepen.io/), [jsfiddle](https://jsfiddle.net/) or [jsbin](https://jsbin.com) to share your example. Please use the [latest un-minified version](https://cdn.plot.ly/plotly-latest.js) of plotly.js in your report unless not applicable._ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the latest points to a really old version ( v1.58.5).
Maybe we could point users to https://github.com/plotly/plotly.js/releases to find the latest released version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch. That's been deprecated for a while. I switched to the releases page as you suggested.
woo hoo! |
Description
Update GitHub issue templates to new format documented here.
Closes #7553.
Changes
Testing
I think that the only test will be opening a new issue once this gets merged to the default branch (master).