diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index ed5d45c62c..039fc7b964 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -1,4 +1,12 @@ - + ## Content diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 29e3b6f366..f0191d43e0 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -10,6 +10,7 @@ * [I want to add new strings to the project](#i-want-to-add-new-strings-to-the-project) * [I want to help translating Element](#i-want-to-help-translating-element) * [Element X Android Gallery](#element-x-android-gallery) +* [I want to add a new feature to Element X Android](#i-want-to-add-a-new-feature-to-element-x-android) * [I want to submit a PR to fix an issue](#i-want-to-submit-a-pr-to-fix-an-issue) * [Kotlin](#kotlin) * [Changelog](#changelog) @@ -76,6 +77,14 @@ Once added to Localazy, translations can be checked screen per screen using our Localazy syncs occur every Monday and the screenshots on this page are generated every Tuesday, so you'll have to wait to see your change appearing on Element X Android Gallery. +## I want to add a new feature to Element X Android + +Thank you for contributing to the project! Please have a look in the [dedicated documentation](./docs/pull_request.md) about pull request. + +Also, please keep in mind that any feature added to Element X Android needs to be added to [the iOS client](https://github.com/element-hq/element-x-ios) too, unless it's related to an Android OS only behaviour. + +**IMPORTANT:** if you are adding new screens or modifying existing ones, this needs acceptance from the product and design teams before being merged. For this, it's better to start with a [feature request issue](https://github.com/element-hq/element-x-android/issues/new?template=enhancement.yml) describing the change you want to make and the motivation behind it instead of directly creating a pull request. This will allow the product and design teams to give feedback on the change before you start working on it, and avoid you doing work that might end up being rejected. + ## I want to submit a PR to fix an issue Please have a look in the [dedicated documentation](./docs/pull_request.md) about pull request. @@ -184,7 +193,7 @@ internal fun PinIconPreview() = ElementPreview { } ``` -This will allow to preview the composable in both light and dark mode in Android Studio. This will also automatically add UI tests. The GitHub action [Record screenshots](https://github.com/element-hq/element-x-android/actions/workflows/recordScreenshots.yml) has to be run to record the new screenshots. The PR reviewer can trigger this for you if you're not part of the core team. +This will allow to preview the composable in both light and dark mode in Android Studio. This will also automatically add UI tests. The GitHub action [Record screenshots](https://github.com/element-hq/element-x-android/actions/workflows/recordScreenshots.yml) has to be run to record the new screenshots. The PR reviewer can trigger this for you if you're not part of the core team. ### Authors