2016-01-18 04:53:10 -08:00
|
|
|
# Contributing Guidelines
|
2016-02-18 07:20:19 -08:00
|
|
|
Thank you for your contributing! My requests are few things. Please read below.
|
2016-01-18 04:53:10 -08:00
|
|
|
|
|
|
|
|
## Issue
|
2016-02-18 07:20:19 -08:00
|
|
|
Thank you for feedback. When you report a problem, please pay attention to following points.
|
2016-01-18 04:53:10 -08:00
|
|
|
|
|
|
|
|
### Does it happen on web browsers? (especially Chrome)
|
2016-02-18 07:20:19 -08:00
|
|
|
electron-mattermost is based on Electron. It integrates Chrome as a browser window.
|
|
|
|
|
If the problem appears on web browsers, it may be the issue for Mattermost (or Chrome).
|
|
|
|
|
|
|
|
|
|
### Try "Clear Cache and Reload"
|
|
|
|
|
It's available as `Ctrl(Command) + Shift + R`.
|
|
|
|
|
Some layout problems are caused by browser cache.
|
|
|
|
|
Especially, this kind of issue might happen when you have updated Mattermost server.
|
2016-01-18 04:53:10 -08:00
|
|
|
|
|
|
|
|
### Write detailed information
|
2016-02-18 07:20:19 -08:00
|
|
|
Following points are very helpful to understand the problem.
|
2016-01-18 04:53:10 -08:00
|
|
|
|
2016-02-18 07:20:19 -08:00
|
|
|
* How to reproduce, step-by-step
|
|
|
|
|
* Expected behavior (or what is wrong)
|
|
|
|
|
* Screenshots (for GUI issues)
|
2016-03-15 04:44:26 -07:00
|
|
|
* Application version
|
|
|
|
|
* Operating system
|
2016-01-18 04:53:10 -08:00
|
|
|
* Mattermost version
|
|
|
|
|
|
2016-03-11 07:22:06 -08:00
|
|
|
## Feature idea
|
|
|
|
|
Please see http://www.mattermost.org/feature-requests/ .
|
|
|
|
|
|
2016-01-18 04:53:10 -08:00
|
|
|
## Pull request
|
2016-03-16 00:04:37 -07:00
|
|
|
Pull requests are welcome. Thank you for your great work!
|
2016-01-18 04:53:10 -08:00
|
|
|
|
2016-03-16 00:04:37 -07:00
|
|
|
1. When you edit the code, please run `npm run prettify` to format your code before `git commit`.
|
|
|
|
|
2. In the description of your pull request, please include:
|
|
|
|
|
* Operating System version on which you tested
|
|
|
|
|
* Mattermost server version on which you tested
|
|
|
|
|
* New or updated unit tests for your changes
|
|
|
|
|
3. Please complete the [Mattermost CLA](http://www.mattermost.org/mattermost-contributor-agreement/) prior to submitting a PR.
|
2016-01-18 04:53:10 -08:00
|
|
|
|
