|
1 | | -I want to merge this change because... |
2 | | - |
| 1 | +### Summary |
3 | 2 | <!-- Please mention all relevant issue numbers. --> |
| 3 | +Main Code: |
| 4 | +``` |
| 5 | +
|
| 6 | +``` |
4 | 7 |
|
5 | | -# Impact |
| 8 | +### Pull Request Checklist |
6 | 9 |
|
7 | | -* [ ] New migrations |
8 | | -* [ ] New/Updated API fields or mutations |
9 | | -* [ ] Deprecated API fields or mutations |
10 | | -* [ ] Removed API types, fields, or mutations |
11 | | -* [ ] Documentation needs to be updated |
| 10 | +<!-- Please keep this section. It will make maintainer's life easier. --> |
12 | 11 |
|
13 | | -# Pull Request Checklist |
| 12 | +1. [ ] Privileged views and APIs are guarded by proper permission checks. |
| 13 | +1. [ ] All visible strings are translated with proper context. |
| 14 | +1. [ ] All data-formatting is locale-aware (dates, numbers, and so on). |
| 15 | +1. [ ] Database queries are optimized and the number of queries is constant. |
| 16 | +1. [ ] Database migration files are up to date. |
| 17 | +1. [ ] The changes are tested. |
| 18 | +1. [ ] The code is documented (docstrings, project documentation). |
| 19 | +1. [ ] GraphQL schema and type definitions are up to date. |
| 20 | +1. [ ] Changes are mentioned in the changelog. |
14 | 21 |
|
15 | | -<!-- Please keep this section. It will make the maintainer's life easier. --> |
| 22 | +### Screenshots |
16 | 23 |
|
17 | | -* [ ] Privileged queries and mutations are guarded by proper permission checks |
18 | | -* [ ] Database queries are optimized and the number of queries is constant |
19 | | -* [ ] Database migration files are up to date |
20 | | -* [ ] The changes are tested |
21 | | -* [ ] GraphQL schema and type definitions are up to date |
22 | | -* [ ] Changes are mentioned in the changelog |
| 24 | +<!-- If your changes affect the UI, providing "before" and "after" screenshots will |
| 25 | +greatly reduce the amount of work needed to review your work. --> |
0 commit comments