|
| 1 | +# Contributing to stream-chat-react |
| 2 | + |
| 3 | +As a contributor, here are the guidelines we would like you to follow: |
| 4 | + |
| 5 | +- [Asking questions](#asking-questions) |
| 6 | +- [Filing an issue](#filing-an-issue) |
| 7 | +- [Contribution Standards](#coding-standards) |
| 8 | +- [Submission Guidelines](#submission-guidelines) |
| 9 | +- [Signing the CLA](#contributor-licence-agreement) |
| 10 | + |
| 11 | + |
| 12 | +## <a name="asking-questions"></a>Are you looking for answers? |
| 13 | +There are many ways you can get your questions answered. It can be hard to decide, where to begin if you are just starting out. We suggest you take a look at the resources in the following order: |
| 14 | + |
| 15 | +### 1. Stream chat API documentation |
| 16 | +Package `stream-chat-react` has a peer dependency [stream-chat-js](https://github.com/GetStream/stream-chat-js) - a client library for interacting with the Stream Chat API (see the [API docs](https://getstream.io/chat/docs/javascript/?language=javascript)). |
| 17 | + |
| 18 | +### 2. Documentation for stream-chat-react |
| 19 | +The [stream-chat-react](https://getstream.io/chat/docs/sdk/react/) documentation is held separately from the Stream Chat API docs. Besides documenting the component API, it provides examples of their use in various scenarios. |
| 20 | + |
| 21 | +### 3. Read the source code |
| 22 | +When you plan on contributing to the repository try to get acquainted with the existing code base. The best way to learn :) |
| 23 | + |
| 24 | +### 4. Take a look at our tutorials |
| 25 | + |
| 26 | + |
| 27 | +### Get help from our Customer Success team |
| 28 | +If what you are looking for is technical support embedding Stream in your application, we suggest emailing our Customer Success team at [email protected] with your application key and the SDK versions you're using. The Issue section of this GitHub repo is now reserved only for bug reports, feature improvements and suggestions. |
| 29 | + |
| 30 | + |
| 31 | +## <a name="filing-an-issue"></a>Filing an issue |
| 32 | +Spotting imperfections and not keeping them to yourself is the first step to make this library better. We are very grateful for reports concerning imperfections in the source code or the [documentation]((https://getstream.io/chat/docs/sdk/react/)). Before filing an issue, please, review the list of [open issues](https://github.com/GetStream/stream-chat-react/issues) first. |
| 33 | + |
| 34 | +### Reporting bugs |
| 35 | +You can report a source code bug by using the [Bug Report template](https://github.com/GetStream/stream-chat-react/issues/new/choose). Make sure you include "steps to reproduce" section. Bug that cannot be reproduced cannot be solved. |
| 36 | + |
| 37 | +Do not be afraid to report imperfections in our [documentation]((https://getstream.io/chat/docs/sdk/react/)) as well. In such case, please attach the `docs` tag to the issue. |
| 38 | + |
| 39 | + |
| 40 | +### Requesting a feature |
| 41 | +You can request a feature by submitting a [Feature request issue](https://github.com/GetStream/stream-chat-react/issues/new?assignees=&labels=feature&template=feature_request.md&title=) in our repository. If you would like to implement the proposal, please state it in the issue. It will allow us to discuss the proposal and better coordinate the efforts. You can even ping us - mention `@GetStream/stream-react-developers ` in the issue. |
| 42 | + |
| 43 | +## <a name="contribution-standards"></a> Contributing to the repo |
| 44 | + |
| 45 | +### Set up for success |
| 46 | +It is always good to get acquainted with the specifics of the package. For example the `stream-chat-react` package has its peer dependencies (`stream-chat-js`, `stream-chat-css`), which you may need to tweak at the same time, while developing the feature for `stream-chat-react`. To get more into those specifics, please read [development guide](./developers/DEVELOPMENT.md). |
| 47 | + |
| 48 | +### Good first issue |
| 49 | + |
| 50 | +###<a name="#coding-rules"></a> Coding rules |
| 51 | +Any contributions to the library should follow Stream's coding rules. |
| 52 | + |
| 53 | +#### 1. Code should be tested |
| 54 | +All the code submitted should be covered by unit tests. Mocking utilities are provided in `src/mock-builders`. Optimally a suite of E2E tests should be included as well. |
| 55 | + |
| 56 | +#### 2. API Changes should be documented |
| 57 | +Changes to components interface exposed to the library integrators should be documented. We keep the documentation `docusaurus/docs/React` folder. |
| 58 | + |
| 59 | +#### 3. Code should be DRY & correctly formatted |
| 60 | +If you find yourself copying source code from one place to another, please extract it into a separate component or function. |
| 61 | + |
| 62 | +#### 4. Keep an eye on performance |
| 63 | +Keep in mind that the chat application may need to work with thousands of messages. |
| 64 | + |
| 65 | +#### 5. Follow commit formatting rules |
| 66 | +We follow [Angular's Commit Message Format rules](https://github.com/angular/angular/blob/master/CONTRIBUTING.md#-commit-message-format) with [possible deviations](./developers/COMMIT.md). The same rules are used by our release automation tool. Therefore, every commit message should strictly comply with these rules. |
| 67 | + |
| 68 | + |
| 69 | +## <a name="submission-guidelines"></a> Submitting your work |
| 70 | +1. Make sure you have signed our Contributor License agreement |
| 71 | +2. [Fork](https://docs.github.com/en/github/getting-started-with-github/fork-a-repo) the repo and create a dedicated git branch locally |
| 72 | +3. Follow the [coding rules](#coding-rules) |
| 73 | +4. Create a descriptive PR ([see more on PR requirements](./developers/PR_REVIEW.md)) |
| 74 | + |
| 75 | + |
| 76 | +## <a name="contributor-licence-agreement"></a> Contributor License Agreement |
| 77 | +Before we can merge your contribution into our repository, we would like to ask you to sign the [Contributor License Agreement](https://docs.google.com/forms/d/e/1FAIpQLScFKsKkAJI7mhCr7K9rEIOpqIDThrWxuvxnwUq2XkHyG154vQ/viewform). |
| 78 | + |
0 commit comments