Feat/detail statistik jaminan sosial#1030
Merged
Merged
Conversation
|
🔄 AI PR Review sedang antri di server...
|
…istik_jaminan_sosial
This was referenced May 7, 2026
Contributor
|
OK |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request: Feature: Detail Statistik Jaminan Sosial
Deskripsi
Menambahkan fitur detail data pada halaman statistik jaminan sosial untuk memungkinkan pengguna melihat data individu dari kategori statistik yang dipilih. Fitur ini mengintegrasikan API eksternal untuk mengambil data detail dan menampilkannya dalam format tabel yang dapat difilter berdasarkan tahun.
Depedenccy
https://github.com/OpenSID/API-Database-Gabungan/pull/397
Perubahan yang dilakukan:
DataPresisiJaminanSosialControlleruntuk menangani logika detail data jaminan sosial dengan sanitasi input dan parameter filteringStatistikJaminanSosialControlleruntuk mengubah detail link dari URL kosong ke endpoint detail data barudetail_data.blade.phpuntuk menampilkan tabel detail data dengan DataTable, filter tahun, dan integrasi APIstatistik-jaminan-sosial.blade.phpdengan JavaScript yang lebih robust untuk handling link detail, filter parameters, dan state management/data-presisi/jaminan-sosial/detail_datadengan middleware permission yang sesuaiDataPresisiJaminanSosialControllerTestuntuk memverifikasi behavior controller dengan berbagai parameter inputAlasan perubahan:
Dampak perubahan:
✅ Fungsionalitas Baru: Pengguna dapat melihat detail individu dari data statistik jaminan sosial
✅ Performance: Penggunaan DataTable dengan server-side processing untuk handling data besar secara efisien
✅ Security: Sanitasi input dan validasi parameter untuk mencegah vulnerability
✅ Maintainability: Kode terstruktur dengan separation of concerns antara controller, view, dan route
Masalah Terkait (Related Issue)
#999
Langkah untuk mereproduksi (Steps to Reproduce)
Sebelum perbaikan (masalah):
Setelah perbaikan (fix):
Testing pada fitur lain yang terkait:
Daftar Periksa (Checklist)
Teknis Detail
Penjelasan Teknis
Fitur menggunakan arsitektur MVC dengan controller untuk business logic, view untuk presentation, dan route untuk URL mapping. Data diambil dari API eksternal melalui AJAX dengan authentication header. DataTable digunakan untuk pagination dan sorting client-side.
Konfigurasi yang berubah
Tidak ada perubahan konfigurasi
Dependencies yang ditambahkan
Tidak ada dependencies baru
Testing
Manual Testing
Automated Testing
Screenshots / Video
Sebelum:
Link pada nilai statistik tidak mengarah ke halaman detail (URL kosong atau tidak valid)
Sesudah:
Link pada nilai statistik mengarah ke halaman detail dengan tabel data individu, filter tahun, dan parameter filtering yang sesuai
Breaking Changes
Tidak ada breaking changes - fitur ini menambahkan fungsionalitas baru tanpa mengubah behavior existing
Migration Guide
Tidak diperlukan - ini adalah penambahan fitur baru
References
/api/v1/data-presisi/jaminan-sosial