- Status changed to Needs review
about 2 years ago 2:17pm 9 February 2023 - @adil_siddiqui opened merge request.
- ๐ฎ๐ณIndia Adil_Siddiqui
@pankaj1390 You tested the patch with php 5.3 and Drupal 7 which will not work as its a drupal 10 issue stated in the problem with php 8.1..So if you want to test the patch please do it with php 8.1 and drupal 10.
- ๐ฎ๐ณIndia Adil_Siddiqui
I think because you earlier created test for 7.x-1.x it should have been for 9.1.x
- ๐ฎ๐ณIndia Manoj Raj.R Chennai
Manoj Raj.R โ made their first commit to this issueโs fork.
- ๐ฎ๐ณIndia Manoj Raj.R Chennai
Added the code and made
MR7 ! as mergeable now and made the rebase so it can merge easily.Please review.
- ๐บ๐ธUnited States wattdesigns
Did anyone make a fix for this? I'm getting it now as well. I've used the module for a long time, but just added ECA to the webpage. Now it's throws this error. I'm using 9.1.0-alpha1+7-dev.
Thanks!