Closed (fixed)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Major
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
19 Aug 2022 at 09:06 UTC
Updated:
9 Nov 2022 at 10:29 UTC
Jump to comment: Most recent
Comments
Comment #2
avpadernoThank you for applying! Reviewers will review the project files, describing what needs to be changed.
Please read Review process for security advisory coverage: What to expect for more details and Security advisory coverage application checklist to understand what reviewers look for. Tips for ensuring a smooth review gives some hints for a smother review.
To reviewers: Please read How to review security advisory coverage applications, What to cover in an application review, and Drupal.org security advisory coverage application workflow.
Since the project is being used for this application, for the time this application is open, only the user who created the application can commit code.
Comment #3
avpadernoComment #4
avpadernoComment #5
heni_deepak commentedonly miner issue in Drupal coding standard
Comment #6
klausimanual review:
config schema for your settings is missing, see https://www.drupal.org/docs/drupal-apis/configuration-api/configuration-...
Otherwise looks good to me!
Thanks for your contribution, Mads!
I updated your account so you can opt into security advisory coverage now.
Here are some recommended readings to help with excellent maintainership:
You can find lots more contributors chatting on Slack or IRC in #drupal-contribute. So, come hang out and stay involved!
Thanks, 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.
Thanks to the dedicated reviewer(s) as well.