Account created on 9 December 2014, over 9 years ago
  • Software engineer at Acquia 
#

Merge Requests

More

Recent comments

🇮🇳India deepakkm

deepakkm changed the visibility of the branch project-update-bot-only to hidden.

🇮🇳India deepakkm

deepakkm changed the visibility of the branch project-update-bot-only to hidden.

🇮🇳India deepakkm

deepakkm changed the visibility of the branch project-update-bot-only to active.

🇮🇳India deepakkm

deepakkm changed the visibility of the branch project-update-bot-only to hidden.

🇮🇳India deepakkm

deepakkm changed the visibility of the branch 3433700-automated-drupal-11 to active.

🇮🇳India deepakkm

deepakkm changed the visibility of the branch 3433700-automated-drupal-11 to hidden.

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

Verified the changes and pipeline is green hence RTBC

🇮🇳India deepakkm

Changes looks good now for D11 compatibility.

🇮🇳India deepakkm

There are few deprecation which should be fixed as part of this ticket

Instantiation of deprecated class                                      
         Drupal\Core\Password\PhpassHashedPassword:                             
         in drupal:10.1.0 and is removed from drupal:11.0.0. The                
           password compatibility service has been moved to the phpass module.  
           Use \Drupal\phpass\Password\PhpassHashedPassword instead. 
🇮🇳India deepakkm

Rebased MR !173 with 8.x-3.x branch. Verified changes locally and looks good to me.

🇮🇳India deepakkm

Verified changes locally and pipeline is green.

🇮🇳India deepakkm

verified changes locally and Pipeline is green.

🇮🇳India deepakkm

deepakkm changed the visibility of the branch project-update-bot-only to hidden.

🇮🇳India deepakkm

Moved all the commit from project-update-bot-only to 3435303-automated-drupal-11.

🇮🇳India deepakkm

@tim - could you please tag the actual ticket for D11.

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

@berdir - There is a separate issue created for fixing these deprecations, kindly have a look at this - https://www.drupal.org/project/default_content/issues/3415901 📌 EntityReferenceTestTrait deprecated Needs review

🇮🇳India deepakkm

Verified changes on local and working fine, hence RTBC

🇮🇳India deepakkm

Remaining phpcs and phpstan issue should be fixed as part of separate ticket. I think MR 16 is enough for drupal 11 comptability

🇮🇳India deepakkm

@chandu7929 - There is a separate issue to fix those deprecations hence i reverted those commit and added it here - https://www.drupal.org/project/default_content/issues/3415901 📌 EntityReferenceTestTrait deprecated Needs review

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

Hello @dave-reid - I have a suggestion on the new release for embed module and was thinking if there is any plan to drop Drupal 9 support as it has ckeditor4 dependency which will not be supported in Drupal 11. Would it be ok if i create an MR with dropped support for Drupal 9?

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

created a new ticket to fix phpunit previous major issue on drupal 9 - https://www.drupal.org/project/password_policy/issues/3449074 📌 PHPunit test failing for Drupal 9 Needs work

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

Verified this patch on Drupal 11 alpha version and its working fine.

🇮🇳India deepakkm

Forgot to add another error when we install site or run drush updb

🇮🇳India deepakkm

@japerry - when we install site with acquia_search 3.2.x-dev we are getting below error and solr core is not getting attached. Kindly have a look at the screenshot below.

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

🇮🇳India deepakkm

deepakkm made their first commit to this issue’s fork.

Production build 0.69.0 2024