New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: Tempat kelahiran salah di Surat Keterangan Kelahiran #901

Closed
balongbesuk opened this Issue Apr 2, 2018 · 11 comments

Comments

5 participants
@balongbesuk
Contributor

balongbesuk commented Apr 2, 2018

sedang_1519876576_rastraii

sedang_1519876576_rastraii

seharusnya Kota / Kabupaten dimana yang Anak tersebut Lahir, Kenapa Jadi Alamat dilahirkan?

dengan begitu isian alamat lahir bisa masuk ke tempat lahir

sedang_1519876576_rastraii

@eddieridwan

This comment has been minimized.

Collaborator

eddieridwan commented Apr 2, 2018

@afri18, @MUHAMMADRANGSEL: harap beri tanggapan. Setuju?

@afri18

This comment has been minimized.

afri18 commented Apr 2, 2018

yang betul TEMPAT LAHIT > bukan ALAMAT LAHIR
TEMPAT LAHIR > Nama Kota atau Kabupaten (bukan kecamtan apalagi desa)

@eddieridwan eddieridwan changed the title from Tentang F.2.01 to Bug: Tempat kelahiran salah di Surat Keterangan Kelahiran Apr 2, 2018

@eddieridwan eddieridwan added the bug label Apr 2, 2018

@eddieridwan eddieridwan added this to DIPRIORITASKAN in Rilis yang sedang dikerjakan Apr 2, 2018

@kodingku

This comment has been minimized.

kodingku commented Apr 3, 2018

yg dirubah apa ya??apakah boleh dicoba

@metalab25

This comment has been minimized.

Contributor

metalab25 commented Apr 3, 2018

sudah saya ubahkan silakan direview
surat_ket_kelahiran.zip

@eddieridwan

This comment has been minimized.

Collaborator

eddieridwan commented Apr 4, 2018

@metalab25 , silakan buat pull requestnya supaya bisa di-review.

@kodingku , mungkin anda bisa pelajari apa yg telah dilakukan @metalab25, sebagai info untuk anda.

Bulan ini kami fokus membantu teman2 agar bisa kontribusi -- termasuk menggunakan pull request, proses baku kontribusi aplikasi open source.

metalab25 added a commit to metalab25/OpenSID that referenced this issue Apr 4, 2018

Add files via upload
Perbaikan issue OpenSID#901

This was referenced Apr 4, 2018

@metalab25

This comment has been minimized.

Contributor

metalab25 commented Apr 4, 2018

sudah saya kirim PR

@eddieridwan eddieridwan moved this from DIPRIORITASKAN to SEDANG DIKERJAKAN in Rilis yang sedang dikerjakan Apr 6, 2018

@eddieridwan

This comment has been minimized.

Collaborator

eddieridwan commented Apr 6, 2018

@metalab25 , maaf belum sempat review. Mudah2an dalam beberapa hari ini.

@eddieridwan

This comment has been minimized.

Collaborator

eddieridwan commented Apr 7, 2018

Ternyata issue ini lebih rumit dari terpikir pada awalnya.

Karena interpretasi yg berbeda pada waktu Surat Keterangan Kelahiran dibuat, saat ini di tabel tweb_penduduk ada kolom alamat_tempat_lahir untuk menyimpan alamat jalan tempat orang itu dilahirkan.

Untuk menerapkan perubahan di issue ini, sesuai dengan petunjuk pengisian F-2.01, kolom alamat_tempat_lahir tidak diperlukan lagi, karena isian Tempat Kelahiran (nama kota atau kabupaten tempat kelahiran) sudah ada kolomnya di tweb_penduduk, yaitu tempatlahir.

Dengan demikian, perubahan yang diperlukan untuk issue ini mencakup:

  1. menghapus kolom alamat_tempat_lahir dari tabel tweb_penduduk. Masukkan penghapusan kolom ini sebagai migrasi di database_model.php
  2. menghapus tampilan kolom alamat_tempat_lahir dari donjo-app/views/sid/kependudukan/penduduk_detail.php
  3. menghapus kolom isian alamat_tempat_lahir dari donjo-app/views/sid/kependudukan/penduduk_form_isian.php
  4. menghapus alamat_tempat_lahir atau mengubah alamat_tempat_lahir dengan tempatlahir di surat/surat_ket_kelahiran/data_form_surat_ket_kelahiran.php, surat/surat_ket_kelahiran/data_rtf_surat_ket_kelahiran.php, surat/surat_ket_kelahiran/f-2.01.php, surat/surat_ket_kelahiran/surat_ket_kelahiran.php
  5. cari semua penggunaan alamat_tempat_lahir lainnya, dan hapus atau sesuaikan seperlunya
  6. tambahkan catatan "(nama kota atau kabupaten kelahiran)" di form surat, dan juga di form isian penduduk, supaya jelas apa yg perlu diisi untuk Tempat Kelahiran

Perlu dicek, pengisian Tempat Kelahiran di form Surat Keterangan Kelahiran mengubah data tempatlahir di tabel tweb_penduduk (dan tampil di semua tampilan penduduk).

@afri18

This comment has been minimized.

afri18 commented Apr 7, 2018

kenapa jadi ruwet ya ? padahal pakai tempat lahir yg di tweb_penduduk aja, lalu tambahkan satu file untuk alamat_dilahirkan
itu menurut saya yg bodoh

@eddieridwan

This comment has been minimized.

Collaborator

eddieridwan commented Apr 7, 2018

Yang ada di rilis sekarang hasil pengertian di issue dulu (beberapa bulan yang lalu) yang di bahas di #632 dan direview teman2. Ya, selama ini tidak ada yg comment. Baru @balongbesuk yg meneliti.

Karena itu, teman2 yg mengerti SELALU diminta memberi spesifikasi yg jelas dan SELALU diminta untuk memeriksa setiap issue yang siap diujicoba.

@eddieridwan

This comment has been minimized.

Collaborator

eddieridwan commented May 5, 2018

Telah dicommit ke master melalui PR #961.

@eddieridwan eddieridwan closed this May 5, 2018

@eddieridwan eddieridwan moved this from SEDANG DIKERJAKAN to SUDAH DI MASTER in Rilis yang sedang dikerjakan May 5, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment