Skip to content
Snippets Groups Projects
Select Git revision
  • main default
  • develop
  • william_feature_getcontent
  • magnus_feature_add-courses-users
  • magnus_feature_add-testing
  • mattias_feature_assignment
  • william_feature_fixgetcontent
  • magnus_feature_content-services-POST-operations
  • magnus_feature_views
  • matwil_feature_tree
  • william_feature_autlogin
  • william_feature_getassignments
  • mattias_feature_setup-project-shell
13 results
Blame History Permalink
PULL_REQUEST_TEMPLATE.md 1.44 KiB

[Short description explaining the high-level reason for the pull request]

Additions

Removals

Changes

Testing

Screenshots

Notes

Todos

Checklist

  • PR has an informative and human-readable title
  • Changes are limited to a single goal (no scope creep)
  • Code can be automatically merged (no conflicts)
  • Code follows the standards laid out in the development playbook
  • Passes all existing automated tests
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future todos are captured in comments
  • Visually tested in supported browsers and devices (see checklist below :point_down:)
  • Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
  • Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist

Browsers

  • Chrome
  • Firefox
  • Safari
  • Internet Explorer 8, 9, 10, and 11
  • Edge
  • iOS Safari
  • Chrome for Android

Accessibility

  • Keyboard friendly
  • Screen reader friendly

Other

  • Is useable without CSS
  • Is useable without JS
  • Flexible from small to large screens
  • No linting errors or warnings
  • JavaScript tests are passing