Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Integrate database with API #23

Merged
merged 24 commits into from
Nov 1, 2018
Merged

Integrate database with API #23

merged 24 commits into from
Nov 1, 2018

Conversation

ebenezerdon
Copy link
Owner

This PR integrates Postgresql database with API endpoints

ebenezerdon and others added 24 commits October 27, 2018 11:39
@ebenezerdon
ch[#161497422] set up database
958600d 
Add migration.js for creating and dropping tables
Add productController.js for productController
Restructure folder structure
@ebenezerdon
ch[#161527584] Add-db-controllers
ee21628
@ebenezerdon
bg[#161529487] Unable-to-post-with-request-body
fed5d3a
@ebenezerdon
bg[#161529487] unable-to-post
54fe455 
req.body returns undefined
refactor functions for update and delete productsController
@ebenezerdon
update products controller to add min value
f4d43a2
@ebenezerdon
ch-add-sales-controller
eb997f4 
add functions for:
getAllSales
getOneSale
addSale
updateSale
deleteSale
@ebenezerdon
ch-add-user-controllers
b24cd91 
add controllers for sales record
add controllers for products
add controllers for users
@ebenezerdon
ch-add-user-controllers
1fd5c70 
add controllers for sales record
add controllers for products
add controllers for users
update user login controller
@ebenezerdon
ch[#161581647] restructure-project
0956672 
delete dummy-models folder
delete dummy-models controller
@ebenezerdon
Merge pull request #19 from ebenezerdon/restructure-project
0450aa9 
ch restructure project
@ebenezerdon
ch[#161581188] add user verification
c28152c 
add input validation
@ebenezerdon
Merge pull request #20 from ebenezerdon/add-user-verification
7bee282 
#161581188 Add user verification and input validation
@ebenezerdon
ch[#161581188] add user verification
59627ee 
add input validation
@ebenezerdon
ch[#161583645] Refactor tests for db
ae83428 
update return message in controllers
update package.json scripts
update .travis.yml
@ebenezerdon
ch[#161624846] Add create user script to .travis.yml
a692e0f
@ebenezerdon
ch[#161624846] Update .travis.yml
bc0d426
@ebenezerdon
ch[#161624846] Update .travis.yml
73ffa5c
@ebenezerdon
ch[#161624846] Update .travis.yml
e9cbd72
@ebenezerdon
ch[#161624846] Update .travis.yml
8a6035f
@ebenezerdon
ch[#161624846] Update .travis.yml
12ff972
@ebenezerdon
Merge pull request #21 from ebenezerdon/setup-travis-database-161624846
c7c65ea 
#161624846 Setup Travis Database
@ebenezerdon
ch[#161633391] Add check for already exists
6fac55d 
add get sale by attendant
add tests for new features
@ebenezerdon
Merge pull request #22 from ebenezerdon/ch-check-exists-161633391
35f094f 
#161633391 Add check for already exists
@ebenezerdon
ch[#161640875] Add auto-input att id in sales
67ad3f6
@ebenezerdon ebenezerdon merged commit 24647f3 into develop Nov 1, 2018
@coveralls
Copy link

Pull Request Test Coverage Report for Build 133

  • 391 of 435 (89.89%) changed or added relevant lines in 13 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-3.9%) to 86.499%
Changes Missing Coverage Covered Lines Changed/Added Lines %
server/models/db.js 2 3 66.67%
server/routes/middleware/validateinput.js 14 16 87.5%
server/routes/controllers/productsController.js 38 45 84.44%
server/routes/controllers/salesController.js 42 53 79.25%
server/routes/controllers/usersController.js 37 60 61.67%
Totals Coverage Status
Change from base Build 99: -3.9%
Covered Lines: 517
Relevant Lines: 564
💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ebenezerdon @coveralls