This repository has been archived by the owner on Dec 11, 2019. It is now read-only.
Update ISSUE_TEMPLATE.md #10920
Merged
Update ISSUE_TEMPLATE.md #10920
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add comments block - Replace list items with headers Closes #10907 Based on mochajs/mocha#2869 Auditors: Test Plan:
luixxiul
commented
Sep 13, 2017
.github/ISSUE_TEMPLATE.md
Outdated
|
|
||
| - Platform (Win7, 8, 10? macOS? Linux distro?): | ||
| For more, check out our community site: https://community.brave.com/ |
There was a problem hiding this comment.
I was going to add a link to our discord channel, which on the second thought seemed too much. If it isn't, I'll add that.
There was a problem hiding this comment.
Not sure if the first line about visiting wiki makes sense at top. I would have it like this
Have you searched for similar issues? There is a lot of feedback and bug reports that we have received and closed as duplicate.
Please search for existing issue via GitHub search or using a search engines.
Please check our community site: https://communit.brave.com and also have a look at our Wiki: https://github.com/brave/browser-laptop/wiki/
There was a problem hiding this comment.
Not sure if the first line about visiting wiki makes sense at top.
I'd agree. I'll edit this comment block.
luixxiul
commented
Sep 13, 2017
| Before submitting this issue, please visit our wiki for common ones: https://github.com/brave/browser-laptop/wiki | ||
| By using search engines along with GitHub search function, you would be able to find duplicates more efficiently. | ||
|
|
||
| For more, check out our community site: https://community.brave.com/ |
There was a problem hiding this comment.
Our Discord community chat is available too. Join us for live discussions: https://discordapp.com/invite/k57tYrS
luixxiul
added
the
priority/P5
bbondy
approved these changes
Sep 19, 2017
|
I think we'll want to tweak it, but I'm ok with this as the new starting point so merging. |
luixxiul
removed
the
priority/P5
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #10907
Based on mochajs/mocha#2869
Auditors:
Test Plan:
Submitter Checklist:
git rebase -ito squash commits (if needed).Test Plan:
Reviewer Checklist:
Tests