The Mailchimp Popup Block module provides a configurable block with a button, that triggers the opening of the Mailchimp Subscriber pop-up.

Similar project

Mailchimp

How they're different

The Mailchimp module provides a mailchimp_signup submodule, that makes it possible to provide a signup form and expose it as a page or block on your page. This signup functionality is driven by the Mailchimp API and currently does not provide a GDPR compliant implementation. The Mailchimp Popup Block provides a configurable block, that can trigger the Mailchimp Subscriber pop-up, which is the recommended GDPR complaint signup method. It does not depend on the Mailchimp API.

Project link

https://www.drupal.org/project/mailchimp_popup_block

Git instructions

git clone --branch 8.x-1.x https://git.drupal.org/project/mailchimp_popup_block.git

PAReview checklist

https://pareview.sh/pareview/https-git.drupal.org-project-mailchimp_popu...

Comments

szeidler created an issue. See original summary.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Issue summary: View changes
sgostanyan’s picture

sgostanyan commented
Status: Needs review » Reviewed & tested by the community

Manual Review

Individual user account
[Yes: Follows] the guidelines for individual user accounts.
No duplication
[Yes: Does not cause] module duplication and/or fragmentation.
Master Branch
[Yes: Follows] the guidelines for master branch.
Licensing
[Yes: Follows] the licensing requirements.
3rd party assets/code
[Yes: Follows] the guidelines for 3rd party assets/code.
README.txt/README.md
[Yes: Follows] the guidelines for in-project documentation and/or the README Template.
Code long/complex enough for review
[Yes: Follows] the guidelines for project length and complexity.
Secure code
[Yes: Meets the security requirements.]
Coding style & Drupal API usage

saesa’s picture

saesa commented
Status: Reviewed & tested by the community » Needs work
  • Your README.md does not follow best practices (headings need to be uppercase). See https://www.drupal.org/node/2181737 .
    • The INTRODUCTION section is missing.
    • The REQUIREMENTS section is missing.
    • The INSTALLATION section is missing.
    • The CONFIGURATION section is missing.
    • szeidler’s picture

    szeidler commented
    Status: Needs work » Needs review

    Thanks for your reviews. I added a more detailed README as reported in #4.

    avpaderno’s picture

    avpaderno
    he/him
    Primary language Italian
    Location Brescia, 🇮🇹 🇪🇺
    commented
    Assigned: Unassigned » avpaderno
    Status: Needs review » Fixed

    Thank 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.

    szeidler’s picture

    szeidler commented

    Thanks for the reviews and @kiamlaluno for the enormous effort you spend in the issue queues!

    Status: Fixed » Closed (fixed)

    Automatically closed - issue fixed for 2 weeks with no activity.