dev: Updated the PR template

This commit is contained in:
lscambo13
2025-05-31 08:17:20 -07:00
parent 19b06a1e1e
commit 4f8e708038
2 changed files with 35 additions and 47 deletions

35
.github/PULL_REQUEST_TEMPLATE.md vendored Normal file
View File

@@ -0,0 +1,35 @@
# Pull Request
Thanks for contributing to ElegantFin! Please review the contributor guidelines before submitting.
## Type of Change
- [ ] Bug fix
- [ ] New feature
## Description
Explain what you changed and why. Include reasoning and goals.
## Screenshots
If applicable, provide before/after images and explain what each shows.
## Cross-platform Testing
Describe how you verified your changes on different devices (desktop, mobile, TV, etc.).
## Test Configuration
- Jellyfin server version:
- Jellyfin client:
- Client browser name and version:
- Device:
## Checklist
- [ ] I performed a self-review of my own code
- [ ] I followed the style conventions (`em` units, minimal media queries).
- [ ] I avoided unnecessary use of `!important`.
- [ ] I commented my code where applicable.
- [ ] I tested my changes on multiple devices, layouts and viewport sizes.

View File

@@ -1,47 +0,0 @@
name: Pull Request
description: Submit a contribution to ElegantFin
title: "[PR] <Short title for your changes>"
labels: []
assignees: []
body:
- type: markdown
attributes:
value: |
Please follow our [Contributor Guidelines](../CONTRIBUTING.md) and keep your PR focused on a single improvement or fix.
- type: textarea
id: description
attributes:
label: Description
description: Describe your changes and the reason behind them.
placeholder: What did you change and why?
validations:
required: true
- type: textarea
id: screenshots
attributes:
label: Screenshots
description: Add before/after images of any visual changes.
placeholder: Drag and drop or paste images here.
validations:
required: false
- type: textarea
id: testing
attributes:
label: Cross-platform Testing
description: Describe how you verified this works across devices (e.g., desktop, mobile, TV).
validations:
required: true
- type: checkboxes
id: checklist
attributes:
label: Checklist
options:
- label: I followed the style conventions (`em` units, minimal media queries).
required: true
- label: I avoided unnecessary use of `!important`.
required: true
- label: I commented my code where applicable.
required: true
- label: I tested my changes across viewports.
required: true