- Issue created by @sidharth_soman
- @sidharth_soman opened merge request.
- Issue was unassigned.
- Status changed to Needs review
almost 2 years ago 12:47pm 20 April 2023 - 🇮🇳India sidharth_soman Bangalore
I have implemented the dependency injection in the above MR. Please review.
- Status changed to Needs work
almost 2 years ago 2:44pm 24 April 2023 - 🇮🇳India Jeya sundhar Coimbatore
Hi @sidharth_soman ,
This merge only contains src/Plugin/PluginEventSubscriber.php , Need to updateviews_migration.services.yml
views_migration.plugin_event_subscriber service arguments.Thank you :)
Jeya Sundharam - Status changed to Needs review
almost 2 years ago 10:54am 25 April 2023 - 🇮🇳India sidharth_soman Bangalore
@Jaya - thanks for the feedback. I've pushed another commit onto the MR for updating the services.yml file. Let me know if this is fine or if you want me to push another MR with both changes in it.
Thanks.
-
Jeya sundhar →
committed 7d076754 on 1.2.x authored by
sidharth_soman →
Issue #3355290 by sidharth_soman, Jeya sundhar: \Drupal calls should be...
-
Jeya sundhar →
committed 7d076754 on 1.2.x authored by
sidharth_soman →
- Status changed to Fixed
almost 2 years ago 6:01am 26 April 2023 Automatically closed - issue fixed for 2 weeks with no activity.