Please make the module ready for drupal 9. Make the changes by removing the deprecated code.

CommentFileSizeAuthor
#7 interdiff_1-7.txt4.24 KBnitesh624
#7 3144281.patch5.01 KBnitesh624
#2 d9-readiness_3144281_1.patch435 bytesmayurgajar

Comments

siddhant.bhosale created an issue. See original summary.

mayurgajar’s picture

mayurgajar commented
StatusFileSize
new d9-readiness_3144281_1.patch435 bytes

This for drupal 9 readiness.. added this patch for temporary on needed basis.

chris matthews’s picture

chris matthews commented
Assigned: siddhant.bhosale » Unassigned
Priority: Normal » Major
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: d9-readiness_3144281_1.patch, failed testing. View results

yang_yi_cn’s picture

yang_yi_cn commented

Error: Call to undefined method Drupal\Component\Utility\Unicode::strtolower() in Drupal\sendgrid_integration\Plugin\Mail\SendGridMail->mail() (line 244

According to https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Component%21Util... ,
the fix is
in drupal:8.6.0 and is removed from drupal:9.0.0. Use mb_strtolower() instead.

nitesh624’s picture

nitesh624
he/him
Primary language English
Location Ranchi, India
commented
Assigned: Unassigned » nitesh624
nitesh624’s picture

nitesh624
he/him
Primary language English
Location Ranchi, India
commented
StatusFileSize
new 3144281.patch5.01 KB
new interdiff_1-7.txt4.24 KB
nitesh624’s picture

nitesh624
he/him
Primary language English
Location Ranchi, India
commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 7: 3144281.patch, failed testing. View results

nitesh624’s picture

nitesh624
he/him
Primary language English
Location Ranchi, India
commented
Assigned: nitesh624 » Unassigned
Status: Needs work » Needs review

  • Perignon committed df8a136 on 8.x-1.x authored by nitesh624 
    Issue #3144281 by nitesh624, mayurgajar: Drupal 9 compatibility
perignon’s picture

perignon commented

Committed.

perignon’s picture

perignon commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.