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.

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

modified this branch to pass continuous integration #16

Merged
merged 1 commit into from
Mar 9, 2019

Conversation

UggoPrince
Copy link
Owner

What does this PR do?

This PR enables the app with json web token authentication of users

Description of Task to be completed?

Have the following endpoints working

POST /api/v1/caterers/auth/signup
POST /api/v1/caterers/auth/login
POST /api/v1/customers/auth/signup
POST /api/v1/customers/auth/login
GET /api/v1/meals
POST /api/v1/meals
PUT /api/v1/meals/id
DELETE /api/v1/meals/id
POST /api/v1/menu
GET /api/v1/menu
POST /api/v1/orders
PUT /api/v1/orders/id
GET /api/v1/orders

How should this be manually tested?

After cloning the repo, cd into it and RUN npm test

  • Using postman test every endpoint above with this header
    key: Content-Type value: application/x-www-form-urlencoded

What is the relevant trello board card?

https://trello.com/c/27gRlRQY/23-users-caterers-and-customers-should-be-able-to-register-and-login-and-get-authenticated

Router.put('/meals/:id', _MealsController.default.modifyMeal);
Router.delete('/meals/:id', _MealsController.default.deleteMeal);
var _default = Router;
exports.default = _default;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'exports' is not defined.


var _express = _interopRequireDefault(require("express"));

var _MealsController = _interopRequireDefault(require("../controllers/MealsController"));

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'require' is not defined.

});
exports.default = void 0;

var _express = _interopRequireDefault(require("express"));

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'require' is not defined.

Object.defineProperty(exports, "__esModule", {
value: true
});
exports.default = void 0;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'exports' is not defined.

@@ -0,0 +1,25 @@
"use strict";

Object.defineProperty(exports, "__esModule", {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'exports' is not defined.

@@ -0,0 +1,23 @@
"use strict";

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the function form of "use strict".

app.listen(port); // , () => console.log(`Example app listening on port ${port}!`));

var _default = app;
exports.default = _default;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'exports' is not defined.

_dotenv.default.config();

var app = (0, _express.default)();
var port = process.env.PORT || 4000;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'process' is not defined.

/* eslint-disable no-console */
_dotenv.default.config();

var app = (0, _express.default)();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bad invocation.


var _menuRoute = _interopRequireDefault(require("./server/routes/menuRoute"));

var _ordersRoute = _interopRequireDefault(require("./server/routes/ordersRoute"));

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'require' is not defined.

@UggoPrince UggoPrince merged commit 9160d71 into develop Mar 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants