Closed (fixed)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
7 Jun 2021 at 09:06 UTC
Updated:
3 Nov 2021 at 11:54 UTC
Jump to comment: Most recent
Comments
Comment #2
keen-on.digital commentedComment #3
avpadernoThank you for applying! Is keen-on.digital a shared account?
Comment #4
keen-on.digital commented@kiamlaluno, Thanks for your comment. No, keen-on.digital is not a shared account.
Comment #5
rajeshreeputraPAreview checklist link, showing no issues.
Thanks,
Rajeshreeputra
Comment #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
avpadernoThat test isn't testing the module, since the module doesn't change the front page. Drupal core already tests the front page in
FrontPageTest.The module should test if its block is rendered correctly, similarly to what
HelpBlockTestdoes.Comment #8
keen-on.digital commented@apaderno, thanks for your comment.
Test has been updated to test the site along with instagram_lite block.
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.
I thank all the dedicated reviewers as well.