- Issue created by @Michelle-Buckby
- Status changed to Needs review
8 months ago 9:30am 17 April 2024 - ๐ฎ๐ณIndia dineshkumarbollu
Hi @Michelle-Buckby
#2 patch fixes the issue, but still i am encountering one error after enabling the module.
TypeError: array_merge(): Argument #1 must be of type array, null given in array_merge() (line 399 of /var/www/html/diet/vb101/web/modules/contrib/content_notify/content_notify.module).
Created patch for both changes
- ๐ฎ๐ณIndia abhishek_virasat
abhishek_gupta1 โ made their first commit to this issueโs fork.
- Merge request !9Issue #3095753 by YesCT: Add another time period for notification for old... โ (Open) created by abhishek_virasat
- Status changed to RTBC
8 months ago 10:35am 17 April 2024 - ๐ฎ๐ณIndia abhishek_virasat
@Michelle-Buckby, #5 patch fixed the issue and I also created MR , moving RTBC++
- ๐ฌ๐งUnited Kingdom Michelle-Buckby
https://www.drupal.org/project/content_notify/issues/3422740 ๐ TypeError: array_merge(): Argument #1 must be of type array, null given in array_merge() RTBC
There's a patch already for the array_merge(), nice to have them both go into 3.0.x in one go though!