Merge pull request #417 from jwillia/readme_changelog_201603

Readme changelog 201603

CHANGELOG.md

@@ -1,20 +1,38 @@
 ##coi-1603
-* Renamed migration files to facilitate migrations being ran in alphabetic order.  The following sql script will fix your existing knex_migrations table if needed.
->update knex_migrations set name = '000000_001.js' where name = '1.js';
->update knex_migrations set name = '000000_002.js' where name = '2.js';
->update knex_migrations set name = '000000_003.js' where name = '3.js';
->update knex_migrations set name = '000000_004.js' where name = '4.js';
->update knex_migrations set name = '000000_005.js' where name = '5.js';
->update knex_migrations set name = '000000_006.js' where name = '6.js';
->update knex_migrations set name = '000000_007.js' where name = '7.js';
->update knex_migrations set name = '000000_008.js' where name = '8.js';
->update knex_migrations set name = '000000_009.js' where name = '9.js';
+* Renamed migration files to facilitate migrations being ran in alphabetic order. New migration files will follow the YYYYMM_xxx.js naming convention. The following sql script will fix your existing knex_migrations table if needed.
+
+    > update knex_migrations set name = '000000_001.js' where name = '1.js'; 
+    
+    > update knex_migrations set name = '000000_002.js' where name = '2.js';
+    
+    > update knex_migrations set name = '000000_003.js' where name = '3.js';
+    
+    > update knex_migrations set name = '000000_004.js' where name = '4.js';
+    
+    > update knex_migrations set name = '000000_005.js' where name = '5.js';
+    
+    > update knex_migrations set name = '000000_006.js' where name = '6.js';
+    
+    > update knex_migrations set name = '000000_007.js' where name = '7.js';
+    
+    > update knex_migrations set name = '000000_008.js' where name = '8.js';
+    
+    > update knex_migrations set name = '000000_009.js' where name = '9.js';
+    >
+
+* Moved bootstrap data needed for the application to run from the seed file into the migrations.  This will make it easier to keep this data up-to-date.
 * Fixed bug where general attachments were not displaying on the admin view.
 * Fixed bug where occasionally questions were not being displayed on the admin detail view.
 * Fixed bug where revise/response screen would work for admins when they had commented on their own disclosure.
 * Added ability to display a warning message if a user answers no to all 'Yes/No' type parent screening questions and an active entity exists.
-* Added ability to display a warning message and block progress if a user answers yes to one or more 'Yes/No' type parent screening questions and no active entity exists
+* Added ability to display a message and block progress if a user answers yes to one or more 'Yes/No' type parent screening questions and no active entity exists
 * Added functionality to update a users disclosure's status from 'Up to Date' to 'Update Needed' if a new project is added for that user, or update a user's disclosure from 'Update Needed' to 'Up to Date' if a user is removed from a project.
+* Added ability for COI Admin to view Additional reviewers who have completed their reviews and the date their review was assigned.
+* Fixed bug where submitted disclosure was editable if user types in URL.
+* Added ability in Review Comments Aggregate view for reviewer and COI admin to see to whom a comments are visible.
+* Fixed issue where the reporter could see the name of COI Admin who made review comment.
+* Made some minor UI adjustments.
+* Began work on notifications, documentation will be provided upon completion.
 
 ##coi-1602
 * Fixed bug on the pi-revise screen which prevented adding a new relation to an existing entity while reviewing.

README.md

@@ -75,21 +75,22 @@ module.exports = {
 ----------
 
 **Step 6**: Apply Migrations:
+
+This will create the tables and insert bootstrap data needed for the application to run.
 ```
 node research-coi/node_modules/knex/lib/bin/cli.js --cwd=db/migration --knexfile <replace with knexfile.js path> migrate:latest --env kc_coi
 ```
-----------
-**Step 7**: Apply Seed Data:
-```
-node research-coi/node_modules/knex/lib/bin/cli.js --cwd=db/migration --knexfile <replace with knexfile.js path> seed:run --env kc_coi
-```
 
-Alternatively you can add some demonstration data to play with:
-```
-node research-coi/node_modules/knex/lib/bin/cli.js --cwd=db/migration --knexfile <replace with knexfile.js path> seed:run --env kc_coi demo
-```
 
-**Step 8**: Configuration Environment Variables:
+
+>######**Step 6.5: Optionally Apply Demo Data**
+> ```node research-coi/node_modules/knex/lib/bin/cli.js --cwd=db/migration --knexfile <replace with knexfile.js path> seed:run --env kc_coi demo```
+>
+> This will add some demonstration data to play with.
+
+----------
+
+**Step 7**: Configuration Environment Variables:
 
 System configuration for COI is done with environment variables. Environment variables can either be set in the system or added on the command line when starting the application. Below is a list of configuration environment variables
 
@@ -180,15 +181,22 @@ System configuration for COI is done with environment variables. Environment var
 > : If your are sure you want to use the auth service over http set this to false.
 > *Default*:  true
 
-**Step 9**: Run Webpack
+-------
+
+>###**Notification Service**
+>
+>In Development
+>
+
+**Step 8**: Run Webpack
 ```
 npm run webpack
 ```
 This may take a few minutes. There will likely be some warnings, but there should no errors.
 
-**Step 10**: Start Up Node
+**Step 9**: Start Up Node
 ```
 DB_NAME=<db_name> DB_PACKAGE=<strong-oracle/mysql> npm start
 ```
 
-**Step 11**: Navigate to hostname:port/coi/
+**Step 10**: Navigate to hostname:port/coi/