chore: update pr template
This commit is contained in:
parent
e87a4f947e
commit
4f09a39929
|
@ -6,6 +6,19 @@
|
|||
|
||||
<!-- List the affected areas (e.g wallet, browser, etc..) -->
|
||||
|
||||
### Developer Checklist
|
||||
|
||||
#### Tests & QA
|
||||
- [ ] I have created user stories for this feature/rework
|
||||
- [ ] I have written unit tests for status-go and QML changes
|
||||
- [ ] I have created a storybook page for all UI components and views
|
||||
- [ ] I have tested this PR on both storybook and the actual client (for UI changes)
|
||||
- [ ] I have followed the architecture guidelines: [Architecture guidelines](../architecture-guide.md)
|
||||
- [ ] For bug fixes, I have written a new unit test case to prevent reintroduction
|
||||
- [ ] I have performed side impact testing for changes in shared components
|
||||
- [ ] A reviewer has Tested the PR during review
|
||||
- [ ] Given enough time for a thorough review
|
||||
|
||||
### Architecture compliance
|
||||
|
||||
- [ ] I am familiar with the application architecture and agreed good practices.
|
||||
|
@ -17,6 +30,11 @@ My PR is consistent with this document: [Architecture guidelines](../architectur
|
|||
|
||||
<!-- screenshot (or gif/video) that demonstrates the functionality, specially important if it's a bug fix. -->
|
||||
|
||||
### QA Testing Requirements
|
||||
|
||||
- [ ] This PR requires QA testing for the entire user story
|
||||
- [ ] This PR is a bug fix and can rely on dev + reviewer testing
|
||||
|
||||
<!-- Uncomment this section for status-go upgrade/dogfooding pull requests
|
||||
|
||||
### Impact on end user
|
||||
|
|
Loading…
Reference in New Issue