Closed (fixed)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
2 Nov 2018 at 06:24 UTC
Updated:
20 Nov 2018 at 20:54 UTC
Jump to comment: Most recent
Comments
Comment #2
prestosaurus commentedComment #3
avpadernoThank you for your contribution!
I am adding the PAReview checklist link. If you haven't done it yet, please check the reported issues, and fix the code as indicated. Don't pay attention to the false positives the checklist could contain.
Next, the reviewers will check the project code, and report here what needs to be changed.
Comment #4
prestosaurus commentedComment #5
prestosaurus commentedComment #6
avpadernoComment #7
prestosaurus commentedThis project has been disabled and I have received no notifications, where can I find details?
Comment #8
avpadernoIt was unpublished from the anti-spam system; I guess the recent edits done from a not confirmed account triggered it. I confirmed your account, so it should not happen again.
Comment #9
prestosaurus commentedMuch appreciated!
Comment #10
prestosaurus commentedAdded options to set a custom image field name. Added option to show both image and name together. Updated php if statements to reduce errors for different field value states.
Comment #11
avpadernoKeep in mind that, since the project is new, you must wait 10 days before opting into security advisory coverage.
Comment #12
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.
I thank all the dedicated reviewers as well.