Skip to content

Henarivan Andhika Abhirama#27

Open
henarivanaa wants to merge 33 commits into
native-fox-2019:masterfrom
henarivanaa:master
Open

Henarivan Andhika Abhirama#27
henarivanaa wants to merge 33 commits into
native-fox-2019:masterfrom
henarivanaa:master

Conversation

@henarivanaa
Copy link
Copy Markdown

@henarivanaa henarivanaa commented Mar 7, 2020

Environment Variable:
PORT=3000
DB_USERNAME=henarivanaa
DB_PASSWORD=wakwaw
JWT_SECRET=wadidaw
CLIENT_ID=385899667070-old78euempf7bfveturipfffc4ijvdip.apps.googleusercontent.com
CLIENT_SECRET=tLWdeJVWz43GxDa6zCYywINU

Fitur tambahan: integrasi CRUD todos user dengan google calendar
Kendala:
Untuk google calendar, masih menggunakan 1 akun gmail. Jadi misalnya menggunakan akun saya (henarivanaa@gmail.com), todo user akan tercatat sebagai event baru di google calendar tapi hanya di akun henarivanaa@gmail.com. Masih butuh waktu buat explore sama nyoba - nyoba nya mbak :D

(edit)
client run di http://localhost:8080, kalo live server kan jadi 127.0.0.1:8080, harus diganti localhost:8080 baru bisa google sign in nya.

Comment thread client/ajax.js
}

const checkProjectTodo = (projectId, todoId, todo, status) => {
console.log(todo)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

ini jangan di push ya

Comment thread client/index.html
<h3>Sign In</h3>
<div class="form-group">
<label>Email address</label>
<input type="email" class="form-control" id="email-login">
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

validasi required yah, atau di fjs function nya

Comment thread client/jquery.js


if (localStorage.getItem("token") !== null) {
$(document).ready(() => {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

kenapa harus ada 2 document ready?

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