SetCron Module

Commit Page
https://drupal.org/node/2282017/commits

Sandbox Page
https://drupal.org/node/2282017

Documentation is on https://www.setcron.com/help#drupal-documentation

Main Site: https://www.setcron.com

----------------------------------------------------------------------------

SetCron allows you to access your scheduled tasks directly on your Drupal admin panel. SetCron works by calling your designated urls periodically according to your settings via an automator provided by https://www.setcron.com.

Features include:

* Full crontab functionality
* Timezone support
* Start/ Stop settings
* Unlimited executions (Run every minute)
* Email Reports
* Failure Notifications
* Execution Statistics

This service is provided for free by SetCron
You will need to signup for an API key.

For more information, please check out https://www.setcron.com.

For documentation, you can refer to https://www.setcron.com/help#drupal-documentation.

Comments

PA robot’s picture

PA robot commented
Status: Active » Needs work

Git clone command for the sandbox is missing in the issue summary, please add it.

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

I'm a robot and this is an automated message from Project Applications Scraper.

SetCron’s picture

SetCron commented
Issue summary: View changes
Priority: Normal » Major
Status: Needs work » Reviewed & tested by the community
SetCron’s picture

SetCron commented
Status: Reviewed & tested by the community » Needs review
davidwhthomas’s picture

davidwhthomas commented
Status: Needs review » Needs work

module .info file is missing required core version compatibility information: https://drupal.org/node/542202#core

joachim’s picture

joachim commented

How does this compare with projects such as:

https://drupal.org/project/ultimate_cron
https://drupal.org/project/cron_control
https://drupal.org/project/elysia_cron

SetCron’s picture

SetCron commented

@davidwhthomas Updated .info file and used drupal_http_request instead of curl library. Commit 94af777 on master

SetCron’s picture

SetCron commented

@joachim SetCron does not depend on user triggered events nor need any access to command line. This is an API module for https://www.setcron.com.

Its usage can be for users on Paas, shared hosting or anyone who prefer to delegate the scheduling of cronjobs to a non command line alternative.

You can also refer to the about page for a summarized comparison.

gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Title: SetCron » [D7] SetCron

Automatic review

There are several errors reported by automated review tools. You really should get these fixed before requesting a manual review. Please see: PAreview for details.

Manual review

Based upon the Project application checklist, this project fails on the following counts:

  • 1.1 Ensure your application contains a repository and project page link. Project does not have a working working "git clone" command / repository link.
  • 2.2 Ensure you are working in a version specific branch. Project is set up to clone "master" branch instead of 7.x-1.x. Please read Moving from a master branch to a version branch.
  • 5.1 Ensure the project page contains detailed information. The project appears to provide a Drupal interface to the following external service: https://www.setcron.com/. It is not clear from the project page that this is the sole(?) purpose of the project. Also see comment #5. You don't fix this by explaining the purpose of the module in a comment in this issue thread. You make sure the project page and README.txt contain all necessary information to understand the purpose of the project, and how it differs from other cron-oriented projects.
  • 5.2 Ensure the repository contains a detailed README.txt. The README.txt file is missing from the repo.
  • 5.3 Ensure the code contains a well-balanced amount of inline-comments. While the module contains comments that looks like they the API documentation and comment standards, the comments are in fact non-standard. For intance the @file tag is missing a one line description.
  • 6.1 Run an automated review and ensure there are no major issues. An automated review reveals major divergences from Drupal's coding standards

In addition it is unclear whether the project owner (SetCron) follows the Drupal.org guidelines for individual user accounts. It looks like a company account to me. You may want to clarify whether this is an individual account or a company account before proceeding.

PA robot’s picture

PA robot commented
Status: Needs work » Closed (won't fix)

Closing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).

I'm a robot and this is an automated message from Project Applications Scraper.