Skip to content

Release 6 Deploy Heroku#9

Open
harynp wants to merge 13 commits into
lively-fox-2017:masterfrom
harynp:master
Open

Release 6 Deploy Heroku#9
harynp wants to merge 13 commits into
lively-fox-2017:masterfrom
harynp:master

Conversation

@harynp
Copy link
Copy Markdown

@harynp harynp commented Oct 16, 2017

No description provided.

@tamatamvan
Copy link
Copy Markdown

sedikit saran, mungkin ada baiknya methods untuk check login dan check admin dipisah file untuk membedakan function module api user yg utama sama yg berupa helpers/middleware.

terus saat update dan juga delete ada baiknya untuk ngasih tau user data mana yg berhasil diupdate atau didelete. jadi bukan cuma 'user deleted aja' response nya. dan akan lebih baik kalau response nya dibungkus ke object biar nanti kalau dah masuk front end lebih gampang.
contoh:

res.send({
  message: 'user deleted',
  deletedUser: {
    username: 'tamvan',
    password: 'sajbdiajsbdijbaskld'.
  }
})

trus hampir sama dgn yg lain, gunakan .env untuk nyimpen teks secret atau salt

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