seyi-adeleke / Docman / 115 / 1

91%

master: 91%

push

travis-ci

Feedback (#31)

* linter(line spaces) :

Lines spacing should be at a maximum of 80 chars

* Unused packages should be removed

* Documentation(slate): The Api documentation should be in a folder called `documentation`

* Delete unused files

* Refactor(helpers) : Helper functions should be exported connsistently

* refactor(tests): Tests were refactored to adequately test new changes to the codebase

* Refactor(linter):

Code was refactored to maintain a proper max length of less than 80 characters

* Refactor(dates):

Dates should be saved in an easily readable format

* Refactor(validation)

Certain edge cases should be handled and return an appropriate response

* Refactor:

Users should be able to get a list of all publicly available documents

* Refactor helpers

* Refactor:

Repipe test coverage

* Bug(hound)

Hound should be ablt to work with eslint

* helpers (validation)

Abstracted the id validation into a helper

* Chore(tests)

Expected tests to return the appropriate response message

* Models

Added Validation to models and migrations

* Bugs(create)

Returns a readable mesage when a user tries to create a document with the wrong access level

Users cannot create multiple documents with same title

When users search for documents the appropriate response is returned

* Bug(roles)

When an admin tries to change the role of a user that doesnt exist an error message is returned

* Documentation

Refactored the documentation to reflect the new responses and added the status codes used in the api

* chore(hound)

Disabled hound

201 of 228 branches covered (88.16%)

Branch coverage included in aggregate %.

343 of 370 relevant lines covered (92.7%)

4.88 hits per line