Saudi Identity Field (Enhancements) Drupal Saudi Sprint #5

Events happening in the community are now at Drupal community events on www.drupal.org.
samaphp's picture
Start: 
2020-12-06 17:30 - 23:00 Asia/Riyadh
Event type: 
Sprint

The sprint challenges (tasks):

  • Replace the t() with dependency injection of $this->t() instead. (NEEDS-WORK) @abdulaziz1zaid
  • We need to address all of the @TODO’s in our extended classes. (Major efforts) (NEEDS-WORK) @saud
  • I don’t think we need hook_field_formatter_info_alter() in .module file. (NEEDS-WORK) @abdulaziz1zaid
  • Make sure schema is supporting multiple values when we fix the multiple values in field settings. (NEEDS-WORK) @3ssom #3185343

We suggested some members to do each challenge for responsibility and organizing purposes, but you may join any of these challenges.

Bonus tasks:
- #3185344 Getting field options from field settings instead of duplicate it.

  • @TODO: Enhance the error message. (NEEDS NEW ISSUE)

  • Collect and translate all phrases and messages. (NEEDS NEW ISSUE)

  • Define hook_requirements to check if the Saudi ID library has been imported and disable the new field. (NEEDS NEW ISSUE) and new description. (new feature)

  • Remove unused TODOs: (Part of this issue)

  • @TODO: We have to show this field and implementing
  • @TODO: Please review this part.
  • @TODO: We have to make sure this value is unique if the settings unique has been selected.

Thanks