Skip to content

Apply more DQA items#28

Merged
estevao90 merged 14 commits into
mainfrom
feat/dqa-items
Feb 16, 2026
Merged

Apply more DQA items#28
estevao90 merged 14 commits into
mainfrom
feat/dqa-items

Conversation

@estevao90
Copy link
Copy Markdown
Contributor

@estevao90 estevao90 commented Feb 16, 2026

Summary

This pull request refactors the migration card UI and status handling to improve clarity, modularity, and user experience. The most significant changes include splitting out migration status displays into dedicated templates, updating the filtering logic to remove the "non-applicable" status from filters and lists, and making various UI/UX improvements to the migration card and its styles.

Type of Change

  • 🐛 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
  • ⚡ Performance improvement
  • ♻️ Code refactoring
  • 🧪 Test improvements
  • 🔧 Chore (build process, dependencies, etc.)

Related Issues

Closes #

Pre-Submission Checklist

Code Quality

  • PHPStan analysis passes: composer test:analysis
  • PHP compatibility check passes: composer compatibility
  • Coding standards pass: vendor/bin/phpcs
  • All tests pass
  • No debug code (var_dump, error_log, etc.) left in production code
  • No commented-out code unless specifically needed for reference
  • Documentation updated for any new features or changed behavior
  • New tests have been added for new functionality
  • All existing tests continue to pass

📖 Read the full contributing guidelines: CONTRIBUTING.md

@estevao90 estevao90 marked this pull request as ready for review February 16, 2026 17:40
@estevao90 estevao90 requested a review from dpanta94 as a code owner February 16, 2026 17:40
@estevao90 estevao90 merged commit 84edb1b into main Feb 16, 2026
12 checks passed
@estevao90 estevao90 deleted the feat/dqa-items branch February 16, 2026 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants