[DDW-893] Update the PR checklist (#2811)

.github/pull_request_template.md

@@ -1,17 +1,35 @@
-This PR CHANGES.
+<!---
+Briefly describe the change.
+-->
+
+This PR ...
 
 ## Todos
 
-- [ ] Todo
+<!---
+Consider creating a TODO list to help others understand the progress of work in a WIP pull request.
+-->
 
-## Screenshots
+- [ ] TODO
 
-- Screenshot
+## Screenshots
 
----
+<!---
+Use the GitHub drag&drop feature to upload default-sized Daedalus window screenshots
+or animated GIFs of important UI changes in both English and Japanese.
+Do not use shadow or any effects. On macOS this can be accomplished the following way:
+1. Use the Command+Shift+4 keyboard shortcut.
+2. Press the Spacebar.
+3. Hold the Option button and click the window you want to capture. 
+-->
 
 ## Testing Checklist
 
+<!---
+Open a thread on #daedalus-qa on Slack, mention `@daedalusqa` and `@daedalusteam`, link the thread below
+-->
+
+
 - [Slack QA thread](https://input-output-rnd.slack.com/messages/GGKFXSKC6)
 - [ ] Test
 
@@ -20,24 +38,18 @@ This PR CHANGES.
 ## Review Checklist
 
 ### Basics
-
-- [ ] PR has been assigned and has appropriate labels (`feature`/`bug`/`chore`, `release-x.x.x`)
+- [ ] PR assigned to the PR author(s) 
+- [ ] `input-output-hk/daedalus-dev` and `input-output-hk/daedalus-qa` assigned as PR reviewers
+- [ ] If there are UI changes, Alexander Rukin assigned as an additional reviewer
+- [ ] PR has appropriate labels (`release-vNext`, `feature`/`bug`/`chore`, `WIP`)
+- [ ] PR link is added to a Jira ticket, ticket moved to In Review
 - [ ] PR is updated to the most recent version of the target branch (and there are no conflicts)
 - [ ] PR has a good description that summarizes all changes
-- [ ] PR has default-sized Daedalus window screenshots or animated GIFs of important UI changes:
-  - [ ] In English
-  - [ ] In Japanese
+- [ ] PR contains screenshots (in case of UI changes)
 - [ ] CHANGELOG entry has been added to the top of the appropriate section (*Features*, *Fixes*, *Chores*) and is linked to the correct PR on GitHub
-- [ ] Automated tests: All acceptance and unit tests are passing (`yarn test`)
-- [ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in *development* build (`yarn dev`)
-- [ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in *production* build (`yarn package` / CI builds)
-- [ ] There are no *flow* errors or warnings (`yarn flow:test`)
-- [ ] There are no *lint* errors or warnings (`yarn lint`)
-- [ ] There are no *prettier* errors or warnings (`yarn prettier:check`)
 - [ ] There are no missing translations (running `yarn manage:translations` produces no changes)
 - [ ] Text changes are proofread and approved (Jane Wild / Amy Reeve)
 - [ ] Japanese text changes are proofread and approved (Junko Oda)
-- [ ] UI changes look good in all themes (Alexander Rukin)
 - [ ] Storybook works and no stories are broken (`yarn storybook`)
 - [ ] In case of dependency changes `yarn.lock` file is updated
 
@@ -55,7 +67,4 @@ This PR CHANGES.
 - [ ] All existing Daedalus Testing scenarios are still up-to-date
 
 ### After Review
-- [ ] Merge the PR
-- [ ] Delete the source branch
-- [ ] Move the ticket to `done` column on the YouTrack board
 - [ ] Update Slack QA thread by marking it with a green checkmark