- Issue created by @smustgrave
- 🇮🇳India pradhumanjainOSL
@smustgrave This issue is closed in this thread. Would you please review.
- 🇮🇳India pradhumanjainOSL
I think we can remove this @todo from these file too
core/includes/theme.inc. - First commit to issue fork.
- Status changed to Needs review
about 1 month ago 5:13am 7 May 2024 - 🇮🇳India kalash-j jaipur
Removed the todo task from the both files. As the function is not deprecated.
- Status changed to Active
about 1 month ago 12:10pm 7 May 2024 - 🇺🇸United States smustgrave
If the other ticket was read would see removing the todos isn’t the solution but to do what the todos said.
- Status changed to Needs review
about 1 month ago 11:43pm 15 May 2024 - 🇮🇳India kalash-j jaipur
Sorry @smustgrave i have miss read the other ticket , i have done those changes as asked it TODO. and Added the TODO comment back to the theme file as changes of that file is already done the that ticket
- Status changed to Needs work
about 1 month ago 2:46pm 16 May 2024 - 🇺🇸United States smustgrave
MR appears to have failures.
How come we are leaving " see https://www.drupal.org/project/drupal/issues/3443759 📌 LinkGenerator.php "System path is deprecated - use the route name and parameters" Needs review ." ? shouldn't that be removed.