The simple_disclaimer module

is a Drupal 7 module which asks authenticated users to acknowledge a disclaimer before accessing content throughout the site.

It differs from existing modules:

  • disclaimer (http://drupal.org/project/disclaimer). This JavaScript (thickbox or jquery) lightbox module doesn't provide for persisting the acknowledgement of the disclaimer as does simple_disclaimer, and is not available for Drupal 7.
  • splash (http://drupal.org/project/splash). This module creates a splash page. It may be adaptable to be used as simple_disclaimer, but serves a different purpose. It is also not available for Drupal 7.

Comments

Elvar’s picture

Elvar commented

Hi Enjaygee

You should be working on a version oriented branch, and not master.
"The most direct documentation on this is Moving from a master branch to a version branch. For additional resources please see the documentation about release naming conventions and creating a branch in git."

Automated review.
http://ventral.org/pareview/httpgitdrupalorgsandboxenjaygee1569400git

Read more about the coding standards her http://drupal.org/node/318

I'm setting this to needs work, will do a manual review when following is fixed. :-)

patrickd’s picture

patrickd commented
Status: Needs review » Needs work

Please don't paste the full automated review in here, it makes the issue look clumsy. A link to the report or attachment is absolutely enough. as you can still edit your comment, please shorten it.
thanks for your participation.

patrickd’s picture

patrickd commented

As installation and usage instructions are quite important for us to review, please take a moment to make your project page follow the tips for a great project page. Also create a README.txt that follows the guidelines for in-project documentation.

We do really need more hands in the application queue and highly recommend to get a review bonus so we can come back to your application sooner.

regards

enjaygee’s picture

enjaygee commented

Thank you very much for the help Elvar. I will make the changes suggested by the automated review and move to an appropriate version-oriented branch.

enjaygee’s picture

enjaygee commented

Thanks for the help patrickd. I will update the project page appropriately and establish a proper README.txt.

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Needs work » Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application.