Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Make the end date in a Date range field (DateRangeItem) optional.
Project link
https://www.drupal.org/project/optional_end_date
Git instructions
git clone --branch 8.x-1.x https://git.drupalcode.org/project/optional_end_date.git
PAReview checklist
https://pareview.sh/pareview/https-git.drupal.org-project-optional_end_d...
Comments
Comment #2
vuilThank you for the contribution!
Please unset
&$table
right afterforeach
because you will receive side-effects without that (optional_end_date.install).__DIR__
should be used insteaddirname(__FILE__)
in optional_end_date.module file.Comment #3
Birk CreditAttribution: Birk at B14 commentedThank you for the review, I've updated the repository.
Commit: 90b0ec77
Comment #4
AkashKumar07 CreditAttribution: AkashKumar07 at OpenSense Labs commentedApplied changes looks good to me. Moving to RTBC.
Thanks
Comment #5
apadernoComment #6
Birk CreditAttribution: Birk at B14 commentedI'm not sure what the "PAReview: single application approval" tag means, and if I need to do anything to proceed?
Comment #7
apadernoThank you for your contribution! I am going to update the project to opt it into security coverage.
These are some recommended readings to help with excellent maintainership:
You can find more contributors chatting on the IRC #drupal-contribute channel. So, come hang out and stay involved.
Thank you, also, for your patience with the review process.
Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.
I thank all the dedicated reviewers as well.
Comment #8
apaderno