Ayu Sudi Dwijayanti #18
Conversation
| # fancy-todo |
There was a problem hiding this comment.
biasanya readme dibuat di sisi server saja, karena sebagai API Documentation
| @@ -0,0 +1,38 @@ | |||
| require('dotenv').config() | |||
There was a problem hiding this comment.
baiknya dibuat kondisi untuk proses development saja
| const authorization = require('../middlewares/authorization') | ||
| const ApiController = require('../controllers/apiController') | ||
|
|
||
| router.post('/create', authentication, TodoController.create) |
There was a problem hiding this comment.
jika di rasa semua routes di bawah ini menggunakan authentication, bisa menggunakan router.use(authentication)
| const moongose = require('mongoose') | ||
| const {Schema} = moongose | ||
|
|
||
| const todoSchema = Schema({ |
There was a problem hiding this comment.
tips: ketika membuat schema, gunakan timestamps: true untuk informasi createdAt dan updatedAt
| } | ||
| }) | ||
|
|
||
| // todoSchema.methods.getDaysLeft = function(){ |
There was a problem hiding this comment.
semoga next bisa diselesaikan
| message: 'Email must include @ and .' | ||
| } | ||
| }, | ||
| password: { |
There was a problem hiding this comment.
validasinya luar biasa yaa :D
|
|
||
|
|
||
| userSchema.pre('save', function () { | ||
| console.log('aaaaaaaaaaaaaaaaaaaaa'); |
| next() | ||
| } | ||
| }) | ||
| .catch(()=>{ |
There was a problem hiding this comment.
jika sudah dibuat default di errorHandler, tidak perlu diinput lagi status dan messagenya
| }) | ||
| } | ||
| }) | ||
| .catch(() =>{ |
There was a problem hiding this comment.
ini catch untuk internal server error yaa
| <div class="beforeLogin"> | ||
| <div class="container" id="register"> | ||
| <div class="registerTodo"> | ||
| <center> |
There was a problem hiding this comment.
?? ini sudah tidak support di HTML5
| $(".beforeLogin").hide() | ||
| $(".regist").hide() | ||
| $(".signIn").hide() | ||
| getAll() |
There was a problem hiding this comment.
perhatikan lagi di proses ini yaa, karena tidak ketrigger ketika berhasil login menggunakan g-signin
Overall sudah OK, pemilihan backgaround bisa lebih smooth themanya supaya tidak bentrok dengan warna element lain. Jika ada perubahan atau deploy, slack instructor yaa |
fitur :
kendala :
tapi gak jadi karena fs nya di script gak bisa require akhirnya
cssnya disempen ke variable (hardcode) lalu dikirim
-belum semuanya ke errorhandler, tapi tidak pakai alert :)
saat dibuka lewat email pada iphone hasilnya bagus..
tapi waktu di buka dari app gmail tidak (tapi ada cssnya tp hasilnya beda)
.env
DEFAULT_PS=Admin123
JWT_KEY=HappyWeekend
GOOGLE_CLIENT_ID=413097102968-5esi6ggnd8o2s2pk1bdjhocuaboaehn8.apps.googleusercontent.com
SENDGRID_API_KEY=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
kekurangan:
saat berhasil create/update/delete tidak ada pemberitahuannya