- Issue created by @codebymikey
- ๐ฎ๐ณIndia abhishek_virasat
abhishek_gupta1 โ made their first commit to this issueโs fork.
- ๐ฎ๐ณIndia abhishek_virasat
@codebymikey, I have fixed the issue and created MR. Please review it once
Hi @abhishek_gupta1, the MR partially addresses the issue, but currently has some coding standards issues as well as making changes to other unaffected parts of the code. The merge request title and commit messages also aren't the most helpful for reviewing, you should try to follow the Drupal commit message format as suggested in the Credit & committing section โ of the page.
I have a fix I'm working on locally that addresses the issue whilst also supporting translated content, so will probably force-push that into the issue fork instead (but since I can't change the merge request title, I might push mine into a separate branch instead). Thanks for attempting to fix the issue though.
codebymikey โ changed the visibility of the branch 3483466-recurring-date-validation to hidden.
The fix for that will be addressed in that issue so as not to conflate issues.
- Merge request !157Issue #3483466 by codebymikey: Recurring date validation targets undefined form elements โ (Open) created by codebymikey
- ๐จ๐ฆCanada mandclu
Thanks for identifying this, and for the work to implement a fix. Merged in.
Automatically closed - issue fixed for 2 weeks with no activity.