88 lines
2.5 KiB
Markdown
88 lines
2.5 KiB
Markdown
# Pull Request
|
|
|
|
## 📋 Summary
|
|
<!-- Provide a brief description of the changes in this PR -->
|
|
|
|
## 🎯 Type of Change
|
|
<!-- Mark the relevant option with an 'x' -->
|
|
- [ ] 🐛 Bug fix (non-breaking change which fixes an issue)
|
|
- [ ] ✨ New feature (non-breaking change which adds functionality)
|
|
- [ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
|
|
- [ ] 📚 Documentation update
|
|
- [ ] 🏗️ Infrastructure/build changes
|
|
- [ ] 🧹 Code cleanup/refactoring
|
|
- [ ] 🧪 Tests only
|
|
|
|
## 🔗 Related Issues
|
|
<!-- Link any related issues using "Fixes #123" or "Related to #123" -->
|
|
|
|
## 🛠 Changes Made
|
|
<!-- Describe the changes made in detail -->
|
|
|
|
### Frontend (Flutter)
|
|
- [ ] UI components updated
|
|
- [ ] State management changes
|
|
- [ ] Navigation changes
|
|
- [ ] New screens/widgets added
|
|
|
|
### Backend (.NET Core)
|
|
- [ ] API endpoints added/modified
|
|
- [ ] Database schema changes
|
|
- [ ] Business logic updates
|
|
- [ ] Authentication/authorization changes
|
|
|
|
## 🧪 Testing
|
|
<!-- Describe the testing done for this change -->
|
|
|
|
### Flutter Testing
|
|
- [ ] Unit tests added/updated
|
|
- [ ] Widget tests added/updated
|
|
- [ ] Integration tests added/updated
|
|
- [ ] Manual testing completed on iOS
|
|
- [ ] Manual testing completed on Android
|
|
|
|
### .NET Testing
|
|
- [ ] Unit tests added/updated
|
|
- [ ] Integration tests added/updated
|
|
- [ ] API testing completed
|
|
- [ ] Database migration tested
|
|
|
|
## 📱 Screenshots/Videos
|
|
<!-- Add screenshots or videos if applicable -->
|
|
|
|
## 📝 Additional Notes
|
|
<!-- Any additional context, warnings, or notes for reviewers -->
|
|
|
|
## ✅ Checklist
|
|
<!-- Mark completed items with an 'x' -->
|
|
|
|
### Code Quality
|
|
- [ ] Code follows the established coding standards
|
|
- [ ] Self-review of the code completed
|
|
- [ ] Code is properly commented (especially complex logic)
|
|
- [ ] No debugging code or console logs left in
|
|
- [ ] Error handling is appropriate
|
|
|
|
### Documentation
|
|
- [ ] Documentation updated (if needed)
|
|
- [ ] API documentation updated (if applicable)
|
|
- [ ] README updated (if needed)
|
|
|
|
### Security & Performance
|
|
- [ ] No sensitive data exposed in code
|
|
- [ ] Performance impact considered
|
|
- [ ] Security implications reviewed
|
|
- [ ] Accessibility guidelines followed (for UI changes)
|
|
|
|
### Testing & Deployment
|
|
- [ ] All tests pass locally
|
|
- [ ] CI/CD pipeline passes
|
|
- [ ] Database migrations work (if applicable)
|
|
- [ ] Feature works in staging environment
|
|
|
|
## 👥 Reviewers
|
|
<!-- Tag specific people if needed -->
|
|
@team-leads @senior-developers
|
|
|
|
---
|
|
**Note**: Please ensure all checkboxes are marked before requesting review. |