PR Review Checklist

PR Review Checklist For Fundamental-NGX:

• Code style (following Guideline)
  • Using proper naming convention
  • Using proper access modifiers
  • Grouping class body elements in proper order
  • No unguarded subscriptions
  • Granularity of code (example)
  • Type-safe code
  • Component API documentation
• For custom input controls
  • Test with async data
  • Test Dirty/Touched states
  • Test number and content of valueChanges ngModelChange events
  • Test with dynamically changing data (like changing Select options in runtime)
• Component properly implementing CssClassBuilder or CssStyleBuilder
• Accessibility - account for keyboard support, screenreader support, proper ARIA attributes, etc.