- Issue created by @AstonVictor
- Assigned to sourabhjain
- First commit to issue fork.
- 🇮🇳India sourabhjain
sourabhjain → changed the visibility of the branch 3418316-injected-dependencies to hidden.
- last update
5 months ago 11 pass - Issue was unassigned.
- Status changed to Needs review
5 months ago 10:52am 1 February 2024 - 🇮🇳India sourabhjain
I have added the dependency injection. Please review the MR 10
- Status changed to Needs work
5 months ago 12:19pm 1 February 2024 - 🇺🇦Ukraine AstonVictor
It still has some \Drupal calls.
I merged Gitlab CI. So, it's easier to view requirements.
- Assigned to sourabhjain
- last update
5 months ago 11 pass - last update
5 months ago 10 fail - last update
5 months ago 10 fail - last update
5 months ago 10 fail - Issue was unassigned.
- Status changed to Needs review
5 months ago 7:22am 2 February 2024 - 🇮🇳India sourabhjain
I have resolved all the \Drupal calls issue from the project. Please review it now.
- last update
5 months ago 11 pass - Status changed to Needs work
5 months ago 8:08am 2 February 2024 - 🇺🇦Ukraine AstonVictor
Hi @sourabhjain,
your code is not correct and throws fatal errors on different pages.
- last update
5 months ago 11 pass - last update
5 months ago 11 pass -
AstonVictor →
committed 93bc5c42 on 8.x-1.x authored by
sourabhjain →
#3418316: Add dependency injection
-
AstonVictor →
committed 93bc5c42 on 8.x-1.x authored by
sourabhjain →
- Status changed to Fixed
5 months ago 8:25am 2 February 2024 Automatically closed - issue fixed for 2 weeks with no activity.