Maulana Ghozi Humama #2
Conversation
|
environment variable : feature tambahan: hambatan : expect yang tidak terwujud :
|
|
testing : |
| @@ -0,0 +1,12 @@ | |||
| { | |||
There was a problem hiding this comment.
ini sebaiknya di ignore, tidak boleh share data credential di github
| @@ -0,0 +1,9 @@ | |||
| PORT=3000 | |||
There was a problem hiding this comment.
ini jg harusnya di ignore, tidak boleh share di github
| } | ||
|
|
||
| const express = require('express') | ||
| const axios = require('axios') |
There was a problem hiding this comment.
kenapa menggunakan axios di sini?
| const morgan = require('morgan') | ||
|
|
||
| mongoose.connect(process.env.BASE_URL, { useNewUrlParser : true, useUnifiedTopology: true, useCreateIndex : true }) | ||
| .then(success => { |
There was a problem hiding this comment.
tidak menerima parameter apa2 di sini
| app.use(morgan('combined')) | ||
|
|
||
| app.get('/', (req, res) => { | ||
| res.send(`Mantappp Connect CCCUUUUKKKKK`) |
|
|
||
| const userSchema = new Schema({ | ||
| name: String, | ||
| email: { |
There was a problem hiding this comment.
lakukan validasi juga apakah email unique atau tidak
| @@ -0,0 +1,10 @@ | |||
| const { varify } = require('../helpers/jwt') | |||
| module.exports = (req, res, next) => { | |||
| try{ | |||
There was a problem hiding this comment.
tips: lakukan pengecekan juga apakah user ada atau tidak di database
|
|
||
| <body> | ||
| <div id="App"></div> | ||
| <script src="https://cdn.jsdelivr.net/npm/sweetalert2@8"></script> |
There was a problem hiding this comment.
ketika menggunakan npm install, tidak perlu lagi menggunakan CDN
| <script src="https://cdn.jsdelivr.net/npm/sweetalert2@8"></script> | ||
| <script src="https://apis.google.com/js/platform.js" async defer></script> | ||
| <script src="https://apis.google.com/js/api:client.js"></script> | ||
| <script src="https://kit.fontawesome.com/5cd1e1d933.js"></script> |
There was a problem hiding this comment.
kalo dari data network ini yang bkin berat
|
ketika menjalankan versi yang sudah di deploy, aplikasi berat dan tidak bisa melakukan apa2(?)
Jika ada perubahan/ tambahan, kabari instructor yaa |
| # Hackticle |
There was a problem hiding this comment.
coba nanti dilengkapi :D
No description provided.