-
Notifications
You must be signed in to change notification settings - Fork 39
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Co-authored-by: ginocingolani <[email protected]>
- Loading branch information
1 parent
6f25626
commit 6ea6c80
Showing
1 changed file
with
19 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
**Screenshots** | ||
|
||
**Related Issue** | ||
E.g: *Closes/Fixes #1234* | ||
|
||
- [ ] **Code Quality** | ||
- [ ] The code follows established coding standards and style guidelines | ||
- [ ] The code is properly commented on and provides context when needed (complex feature or hard to understand) | ||
- [ ] **Functionality validation** | ||
- [ ] Feature working as described in the related issue | ||
- [ ] Does this PR need environment variables to be set? If yes, have they been set? | ||
- [ ] New API endpoints are properly documented | ||
- [ ] New functions are properly tested (When should we add tests?) | ||
- [ ] Code errors are being logged in the code health service | ||
- [ ] **Product validation** | ||
- [ ] UI changes look the same as in Figma | ||
- [ ] Usage events are being sent to the product analytics service | ||
- [ ] New common components have been documented in Storybook | ||
- [ ] Preview link if it’s a customer facing issue |