nafiesl / silsilah

A genealogy/family tree application, built with Laravel.
MIT License
590 stars 285 forks source link

[Proposal] Feature: Delete User #10

Closed penjamoen closed 6 years ago

penjamoen commented 6 years ago

apakah ada menu untuk hapus data kalau misal terjadi kesalahan ?

nafiesl commented 6 years ago

Saat ini belum ada tombol untuk hapus User mas @penjamoen. Pertimbangannya, jika ada user yang sudah memiliki keturunan dihapus, maka jalur silsilah bisa putus. Masih belum ketemu formula yang pas untuk menghapus user tanpa memutus jalur silsilah.

Issue ini kita beri label sebagai enhancement ya.

penjamoen commented 6 years ago

mungkin perlu di tambahkan di bagian tambah istri / tambah suami keterangan atau status nya . misal nya sudah cerai . karena disitu belum ada keterangannya jadi sepertinya wanita nya mempunyai 2 suami . terimakasih.

nafiesl commented 6 years ago

Kalau status cerai itu beda topik lagi mas, sebaiknya dibuatkan issue baru sebagai proposal fitur baru. Agar kita bisa keep track setiap usulan dari teman-teman pengguna project ini.

Terima kasih mas @penjamoen.

ad3n commented 6 years ago

@nafiesl : kalau misal sebelum hapus migrasi silsilah atau istilahnya reassign bisa ga ya? Jadi pas klik hapus kalau udah ada relasi maka muncul pop up untuk mengganti ybs dengan orang lain...

nafiesl commented 6 years ago

nah setuju mas @ad3n, berarti ada opsi untuk memilih user pengganti dulu ya? Jadi prosesnya ketika akan hapus user:

  1. Muncul semacam popup atau modal ldialog pilihan user
  2. User pengganti dipilih
  3. Klik tombol "Replace and Delete User"
  4. Sistem mengganti id user di semua record yang berhubungan
  5. Data user dihapus dari aplikasi.

Mantap idenya mas Sur. Insyaallah saya coba terapkan untuk improvement ini.

ad3n commented 6 years ago

Iya mas kira-kira begitu... hehehe semangat mas, luar biasa projectnya

nafiesl commented 6 years ago

Terima kasih banyak mas Sur untuk idenya dan apresiasinya. Senang sekali dapat kontribusi ide dari mas Surya :+1:

nafiesl commented 6 years ago

Mas @penjamoen, mas @ad3n. Fitur delete user dan pemilihan pengganti sudah selesai. :+1:

Kalau mas ada waktu senggang, mungkin bisa git pull dan bantu review ya. :smiley: Terima kasih banyak mas-mas.

shutter_20180601-005 shutter_20180601-006

ad3n commented 6 years ago

Luar biasa... Mantab

penjamoen commented 6 years ago

siap mas .