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

AbrarNa'im #7

Open
wants to merge 34 commits into
base: master
Choose a base branch
from
Open

Conversation

abrarnaim015
Copy link

No description provided.

@abrarnaim015 abrarnaim015 changed the title sedang proses validasi AbrarNa'im Oct 26, 2020
@@ -0,0 +1,99 @@
const {Todo} = require ('../models')
const Super = require('../helper/super.js')
Copy link
Contributor

Choose a reason for hiding this comment

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

nama class nya jangan Super ya, mungkin InputValidation lebih cocok

@dhimashary
Copy link
Contributor

untuk validasi tanggal tidak bisa menggunakan isAfter karena argsnya jalan ketika server dijalankan, misal server nyala di 17 agustus , di 19 agustus kita masih bisa bikin todo untuk 18 agustus (karena new date nya dibuat saat 17 agustus)

@dhimashary
Copy link
Contributor

need improvement:

pembuatan model user, validasi email format, validasi email unique
validasi title, description, status, duedate tidak boleh kosong/null

let dataBody = req.body
console.log(dataBody, 'benar' )
let newData = Super.validasiRegister(dataBody.password)
let dataUpuser = {
Copy link
Contributor

Choose a reason for hiding this comment

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

penamaan variabel dibenahi ya, misal newUser

routers.get('/', TodoController.getTodo)
routers.post('/', TodoController.postTodo)

routers.use(authorization)
Copy link
Contributor

Choose a reason for hiding this comment

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

ini tidak bisa begini karena authorization butuh params id


routers.use(authorization)

routers.get('/:id', TodoController.getTodoById)
Copy link
Contributor

Choose a reason for hiding this comment

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

authorization di selipin aja ke tengah routers yg butuh authorization

@dhimashary
Copy link
Contributor

di navbar yang ga dipake tolong dihapus ya navigasinya

@dhimashary
Copy link
Contributor

google login masih belum bekerja

client/index.js Outdated
}
})
.done(res=>{
console.log(res)
Copy link
Contributor

Choose a reason for hiding this comment

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

karena hanya di console log jadi belom jalan ya google sign in nya

jikanjs.settings.setBaseURL('https://api.jikan.moe/v2'); // sets the API Base URL
jikanjs.settings.setBaseURL('https://api.jikan.moe/v2', 2); // sets also the api version

class AnimeController{
Copy link
Contributor

Choose a reason for hiding this comment

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

data anime belum dipakai

@dhimashary
Copy link
Contributor

dokumentasi belum update

@dhimashary
Copy link
Contributor

baiknya ketika delete user dikonfirmasi yakin/nggak mau delete datanya

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