refactor : dependencies, notifications & instant run

refactor : dependencies, notifications & execute

Author: roshangautam

CONTRIBUTING.md

@@ -1,6 +1,6 @@
-# Studio Contribution Guide
+# Code Studio Contribution Guide
 
-This page contains guidelines for contributing to the Studio packages. Please review these guidelines before submitting any pull requests to the package.
+This page contains guidelines for contributing to the Code Studio packages. Please review these guidelines before submitting any pull requests to the package.
 
 ## Which Branch?
 
@@ -18,10 +18,4 @@ If you have an idea for a new feature you would like to see added to the package
 
 ## Coding Guidelines
 
-Studio follows the [PSR-1](https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-1-basic-coding-standard.md) and [PSR-4](https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-4-autoloader.md) coding standards. In addition to these standards, below is a list of other coding standards that should be followed:
-
-- Namespace declarations should be on the same line as `<?php`.
-- Class opening `{` should be on the same line as the class name.
-- Function and control structure opening `{` should be on a separate line.
-- Interface names are suffixed with `Interface` (`FooInterface`)
-- Use tabs instead of 4 spaces.
+Code Studio follows the [PSR-1](https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-1-basic-coding-standard.md) and [PSR-4](https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-4-autoloader.md) and laravel coding standards.

README.md

@@ -78,6 +78,22 @@ All artisan commands can be scheduled. If you want to hide a command from Totem
 protected $hidden = true;
 ```
 
+##### Use one of the following options if you are registering your commands in app/Console/Kernel in L5.4 and below 
+
+- Option 1 : Create a array variable, let's say `$artisanCommands` in your app/Providers/AppServiceProvider and list all your commands just like you would do in app/Console/Kernel class. Now in the register method add the following
+
+```
+$this->commands($this->artisanCommands);
+```
+
+- Option 2: Roll your own ConsoleServiceProvider in app/Providers, create a array variable, let's say `$artisanCommands` and list all your commands just like you would do in app/Console/Kernel class. Now in its register method add the following. Don't forget to add this new provider to config/app.php's providers array.
+
+```
+$this->commands($this->artisanCommands);
+```
+
+From L5.5 onwards all commands are auto registered, so this wouldn't be a problem.
+
 ### Screenshots
 
 ##### Task List

composer.json

@@ -16,14 +16,22 @@
     }
   ],
   "require": {
-    "php": ">=5.6.4"
+    "php": ">=5.6.4",
+    "illuminate/bus":"~5.4.0",
+    "illuminate/console": "~5.4.0",
+    "illuminate/contracts": "~5.4.0",
+    "illuminate/database":"~5.4.0",
+    "illuminate/events": "~5.4.0"
   },
   "require-dev": {
     "mockery/mockery": "~0.9",
     "orchestra/database": "~3.0",
     "orchestra/testbench" : "~3.0",
     "phpunit/phpunit": "~6.0"
   },
+  "suggest": {
+    "nexmo/client": "Required for sms notifications."
+  },
   "autoload": {
     "psr-4": {
       "Studio\\Totem\\": "src/",

database/migrations/2017_08_26_083622_alter_tasks_table_add_notifications_fields.php

@@ -0,0 +1,34 @@
+<?php
+
+use Illuminate\Support\Facades\Schema;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Database\Migrations\Migration;
+
+class AlterTasksTableAddNotificationsFields extends Migration
+{
+    /**
+     * Run the migrations.
+     *
+     * @return void
+     */
+    public function up()
+    {
+        Schema::table('tasks', function (Blueprint $table) {
+            $table->string('notification_phone_number')->nullable()->after('notification_email_address');
+            $table->string('notification_slack_webhook')->nullable()->after('notification_phone_number');
+        });
+    }
+
+    /**
+     * Reverse the migrations.
+     *
+     * @return void
+     */
+    public function down()
+    {
+        Schema::table('tasks', function (Blueprint $table) {
+            $table->dropColumn('notification_phone_number');
+            $table->dropColumn('notification_slack_webhook');
+        });
+    }
+}