Needs review
Project:
OneAll Social Login
Version:
8.x-2.89
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
6 Aug 2019 at 07:57 UTC
Updated:
14 Jan 2021 at 06:43 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
sergiu stici commentedHere is the patch, please review.
Comment #3
ClaudeSchlesser commentedHello,
thank you very much for the patch, looks great ;)
What I do not like so much is that Drupal::messenger requires Drupal 8.5 and our goal is to have a Drupal 8.x compatible extension. I will have a closer look at your patch, maybe I can change it a bit ;)
Comment #4
rokzabukovec commentedI created my own patch with deprecation and coding standard fixes. Upgrade status and drupal-check analysis show that now the module is ready for Drupal 9. Please review the patch.
Best regards.
Comment #5
eduardo morales albertiRepatch to the last version using the module drupal rector, upgrade status and a clean drupal 9 installation.
Comment #6
muaz7731try out patch from #5 having problem when patching social_login.info.yml
edit:
quick fix just add below "core: '8.x'" on social_login.info.yml
core_version_requirement: ^8 || ^9