Closed (fixed)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
13 Jul 2021 at 17:18 UTC
Updated:
5 Oct 2021 at 07:44 UTC
Jump to comment: Most recent
Comments
Comment #2
anatolii1309 commentedComment #3
anatolii1309 commentedComment #4
avpadernoComment #5
avpadernoComment #6
avpadernoI changed the issue priority as described on Review process for security advisory coverage: What to expect / Application Review Timelines.
To reviewers: Please change back the priority to Normal after reviewing the project.
Comment #7
avpadernoA service class doesn't implement that method, as Drupal reads the .services.yml file to understand which parameters need to be passed to its constructor.
Comment #8
anatolii1309 commentedThanks for your review. I have corrected the code.
Comment #9
avpadernoThank you for your contribution! I am going to update your account.
These are some recommended readings to help with excellent maintainership:
You can find more contributors chatting on the IRC #drupal-contribute channel. So, come hang out and stay involved.
Thank you, also, for your patience with the review process.
Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.