kelnaj [kelnaj]

Hello, and thanks for applying for a CVS account. I am adding the review tags, and some volunteers will review the code, pointing out what it needs to be changed.

As per requirements, the motivation message should be expanded to contain more features of the proposed project. For themes, it should include also a screenshot of the theme, and (when possible) a link to a working demo site using the proposed theme; for modules, it should include also a comparison with the existing solutions.

+1 subscribe

would be great to have this module, i will try it on one of my productions-sites ;)

ok, first things: on settings-page i cant change the version, it always switchs back to for example IE6, i want to notify everthing until IE10...

on IE6 the banner doesn't show at all...

The module's js is not using Drupal.behaviours.

Also, this function seems unnecessary.

function checkIfFalse(checkvar){

  if(checkvar == "false" || checkvar == "0") return false;

  return true;

}

Overall the js file is not eligible to be contributed, this and the fact that it has been 7 weeks without response, is why I'm setting this to won't fix.

CVS maintainer stil needs to have his say =)

I agree that a single JavaScript file doesn't make a module. We need more code to approve an application.

Please read the following links as this is very important information about CVS applications.

• Drupal.org has moved from CVS to Git! This is a very significant change for the Drupal community and for these applications.
• Please read Migrating from CVS Applications to (Git) Full Project Applications and Applying for permission to opt into security advisory coverage on how this affects and benefits you and the application process. In short, every user has now the permissions necessary to create new projects, but they need to apply for opt into security advisory coverage. Without applying, the projects will have a warning on projects that says:
  

  This project is not covered by Drupal’s security advisory policy.