Skip to content

Commit

Permalink
Add PR template to remind us to perform a number of checks
Browse files Browse the repository at this point in the history
  • Loading branch information
barryo committed Jul 16, 2017
1 parent 94ebc65 commit d1aa477
Showing 1 changed file with 15 additions and 0 deletions.
15 changes: 15 additions & 0 deletions docs/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
*PR template - remove this line and edit below*

[BF] Summary of fix - fixes [inex|islandbridgenetworks]/IXP-Manager#x

[NF] New feature summary - closes [inex|islandbridgenetworks]/IXP-Manager#x

*Longer description*


In addition to the above, I have:

- [ ] ensured all relevant template output is escaped to avoid XSS attached with `<?= $t->ee( $data ) ?>` or equivalent.
- [ ] ensured appropriate checks against user privilege / resources accessed
- [ ] API calls (particular for add/edit/delete/toggle) are not implemented with GET and use CSRF tokens to avoid CSRF attacks

0 comments on commit d1aa477

Please sign in to comment.