Add Delete User and Jobs Information

[vickyKDV]

Add Delete user & Jobs Information

[nafiesl]

mas @vickyKDV, terima kasih untuk PR nya.

Tapi bisa kita lihat di sini, perubahan di PR ini belum lolos Travis CI (automated testing), artinya ada bug di sini mas. Silakan mas perbaiki dulu ya.

Btw sekalian saya sampaikan :

• Tidak boleh ada testing yang fail
• Pertimbangkan jika user yang dihapus ini sudah punya pasangan (user_id nya ada di tabel couples)
• Pertimbangkan jika user yang dihapus ini sudah diset sebagai ayah/ibu (masuk di father_id atau mother_id user lain)
• Pertimbangkan jika user yang dihapus ini menjadi pengelola user lain (masuk sebagai manager_id di table users)
• Pertimbangkan jika user yang dihapus ini menjadi pengelola pasangan (masuk sebagai manager_id di table couples)
• Sangat disarankan ada automated testing untuk fitur hapus user ini, tapi ini optional saja, ini bisa saya bantu tambahkan).

[vickyKDV]

1. Untuk yang fail saya sedang check untuk bug nya mas.
2. Untuk Father,Mother,Husban dan Lainnya yang menyangkut user Id sepertinya saya lebih baik pakai Trigger mySQL saja mungkin ya, cuma saya tidak bisa memasukan coding trigger ke Migrate nya mas..

[nafiesl]

Di repo ini kita ga pakai mysql trigger mas, itu ga masuk di source code, jadi jika user dihapus, sistem langsung :

1. Set null pada father_id dan mother_id, atau manager_id user yang berhubungan,
2. detach pasangan (artinya hapus record couple nya),
3. set null parent_id (jika point 2 terjadi).

[nafiesl]

mas @vickyKDV, sepertinya update Pull Request ini diluar dari yang kita diskusikan sebelumnya (hanya fitur delete user saja). Di sini saya lihat ada tambahan:

1. Kolom job dan job_description di table user
2. Lokasi upload Foto berubah ke direktori public

Yang menurut pertimbangan saya saat ini belum diperlukan.

Jadi mohon maaf mas, saya belum bisa merge Pull Request ini. Untuk pengembangan versi mas sendiri, saya persilakan mas buat pada repo github mas vicky sendiri ya.

Terima kasih mas @vickyKDV

[vickyKDV]

baik, mas @nafiesl , saya minta maaf sebelumnya karna memang ini diluar diskusi.