Before approving and merging a pull request, verify:
Content and presentation
- Page preview renders correctly
- Content follows the style guide
- No copyright issues
- Contributors acknowledged and gave permission
Metadata
For resource pages
Verify required resource metadata fields (see adding resources guide):
- Unique
id(use kebab-case) titleprovidedresourceUrllinks to the actual resourcedescriptionclearly explains the resourceobjectivedescribes what the resource aims to achievecontributorslisted and added to CONTRIBUTORS.ymlcoordinatorslisted (must also be in contributors)contactsprovided with valid email addressestagsuse existing tags from resource_tags.yml (3-7 tags recommended)logofile added toassets/img/logos/if providedscreenshotsadded toassets/img/screenshots/if provided
For other pages
Verify required fields (see page metadata guide):
- Unique
page_id(check existing IDs) contributorslisteddescriptionprovidedsearch_excluderemoved (unless intentionally excluded)related_pagesconfigured if applicabletrainingmaterials added if applicableaffiliationspecified if applicable
Navigation and linking
- Page linked in appropriate sidebar menu
- Contributors added to CONTRIBUTORS.yml with contact info
- Pull request linked to related issues
- No merge conflicts with main branch
Review process
- Requested changes have been implemented
- Contributors thanked and informed about publication timeline
- All conversations resolved
Final step
The editor providing final approval should merge the pull request.