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

Andhika Muttaqien #32

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Conversation

andhikam
Copy link

@andhikam andhikam commented Dec 1, 2020

No description provided.

@andhikam
Copy link
Author

andhikam commented Dec 1, 2020

Commit Ke-1 :

Server done (CRUD Task, register, login, google-login)
Client homepage layout done


#### \*Updated 01 Dec 2020, 16:42AM GMT+07:00

#### \*API-Doc with MD on Progress
Copy link
Contributor

Choose a reason for hiding this comment

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

oke on progress ya untuk markdown nya

server/app.js Outdated
@@ -0,0 +1,20 @@
require('dotenv').config();
Copy link
Contributor

Choose a reason for hiding this comment

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

ini jgn lupa di kasih conditional ya

@@ -0,0 +1,33 @@
const { verifyToken } = require('../helpers/jwt');
Copy link
Contributor

Choose a reason for hiding this comment

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

authentication aman

@@ -0,0 +1,14 @@
const jwt = require('jsonwebtoken');
Copy link
Contributor

Choose a reason for hiding this comment

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

jwt aman

@@ -0,0 +1,16 @@
const bcrypt = require('bcryptjs');
Copy link
Contributor

Choose a reason for hiding this comment

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

bcrypt aman

@@ -0,0 +1,44 @@
module.exports = (err, req, res, next) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

nice error handler lumayan lengkap

Comment on lines 6 to 24
router.post('/register', UserController.register);

router.post('/login', UserController.login);

router.post('/googleLogin', UserController.googleLogin);

router.use(authentication);

router.post('/tasks', TaskController.create);

router.get('/tasks', TaskController.getAllTasks);

router.get('/tasks/:id', TaskController.getTask);

router.patch('/tasks/:id', authorization, TaskController.updateTask);

router.put('/tasks/:id', authorization, TaskController.editTask);

router.delete('/tasks/:id', authorization, TaskController.deleteTask);
Copy link
Contributor

Choose a reason for hiding this comment

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

good

@andhikam
Copy link
Author

andhikam commented Dec 2, 2020

Client :

Login & Register done with vue & axios,
Transition page done with vue.

@andhikam
Copy link
Author

andhikam commented Dec 3, 2020

Commit Ke - 3 & 4 :

Finalize API Doc on Markdown
Deploy Server
Add category on Edit Task,
Revise message errorHandler

@andhikam
Copy link
Author

andhikam commented Dec 3, 2020

Commit Ke-5 :

CRUD Client, Login, Register, Fetching Task Done,
Revise Server, adding Category Model,
Vue Component Done

@andhikam
Copy link
Author

andhikam commented Dec 4, 2020

Commit Ke-6 :

Finalizing Client
Server Deploy - Done - https://kanban-app-amp.herokuapp.com
Client Deploy - Done - https://qanban-app.web.app
API Documentation - Done

@andhikam
Copy link
Author

andhikam commented Dec 4, 2020

Commit Ke-7 :

Finalize Client Styling

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.

2 participants