Skip to content

Hilangkan menu seeder yang tidak memiliki sumber#1658

Merged
analisopendesa merged 3 commits into
rilis-devfrom
dev-1650
Jul 16, 2026
Merged

Hilangkan menu seeder yang tidak memiliki sumber#1658
analisopendesa merged 3 commits into
rilis-devfrom
dev-1650

Conversation

@habibie11

@habibie11 habibie11 commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

issue # #1650

🎯 Deskripsi

Pull request ini menyelesaikan masalah terkait beberapa menu bawaan di website (baik pada frontend maupun di dashboard Pengaturan Menu) yang tidak memiliki sumber halaman aktif, sehingga dapat dianggap sebagai bug atau membingungkan pengguna PKS.

Berdasarkan laporan pada issue, menu Desa dan Kategori Berita muncul secara bawaan tanpa halaman yang valid. Selain itu, menu FAQ mengarah pada URL absolute (https://demodk.opendesa.id/faq) alih-alih URL relative (/faq).

PR ini membersihkan seeder menu yang sudah usang (das_navigation) serta menambahkan migration baru untuk memperbaiki data existing pada tabel menu dinamis (nav_menus), guna memastikan konsistensi antarmuka navigasi baik pada instalasi baru maupun sistem yang sudah berjalan.


🛠️ Perubahan yang Dilakukan

1. database/seeders/DasNavigationTableSeeder.php & database/seeders/Demo/DemoDasNavigationTableSeeder.php

Fix — Pembersihan Menu Statis (Tabel das_navigation):

  • Menghapus entri menu parent "Desa" dari daftar $menuUtama dan menyusun ulang urutan (order) untuk menu-menu setelahnya (Potensi, Statistik, Unduhan, FAQ).
  • Mengubah rute untuk menu FAQ yang sebelumnya dirender menggunakan url('faq') (URL absolute) menjadi relative path 'faq'.
  • Menghapus logika dinamis (penggunaan ->each()) yang menambahkan sub-menu ke dalam parent "Desa" baik di mode standar maupun mode Demo, untuk mencegah error relasi ke data yang tidak ada.

2. database/migrations/2026_07_13_155816_fix_nav_menus_data.php

Fix — Koreksi Menu Dinamis (Tabel nav_menus):

  • Menambahkan migration baru untuk menghapus menu "Kategori Berita" (id: 10) dan "Desa" (id: 11) dari pengaturan menu (dashboard).
  • Memperbarui tautan menu FAQ (id: 24) dari sebelumnya berupa URL statis https://demodk.opendesa.id/faq menjadi /faq dengan target _self dan tipe modul.

✅ Test Cases yang Diimplementasikan

  • Instalasi baru (php artisan migrate:fresh --seed) berjalan tanpa ada error akibat penghapusan parent node "Desa".
  • Proses seeding Demo (php artisan db:seed --class=DemoDatabaseSeeder) selesai tanpa error.
  • Menu "Desa" tidak lagi muncul di navigasi frontend (jika menggunakan struktur lama).
  • Halaman Pengaturan Menu di Dashboard (/setting/nav-menu) tidak lagi memuat menu "Desa" dan "Kategori Berita".
  • URL untuk menu FAQ telah menggunakan format relative path /faq baik di frontend maupun di dashboard Pengaturan Menu.

📸 Cara Menjalankan Uji Coba Manual

  1. Jalankan php artisan migrate untuk mengaplikasikan perbaikan data menu pada pengaturan lokal Anda.
  2. Login dan masuk ke halaman Pengaturan > Menu (/setting/nav-menu).
  3. Verifikasi bahwa menu "Kategori Berita" dan "Desa" sudah tidak ada di dalam struktur menu.
  4. Periksa pengaturan pada menu FAQ dengan menekan tombol Edit (ikon pensil).
    • Seharusnya: URL bernilai /faq, dengan Tipe modul dan Target _self.
  5. (Opsional) Jalankan perintah php artisan db:seed --class=DasNavigationTableSeeder.
  6. Akses halaman publik, dan pastikan menu FAQ mengarah pada rute relatif yang benar.

📸 Screenshot atau Video

image

⚠️ Catatan Penting

Karena OpenDK memiliki tabel statis (das_navigation) dan dinamis (nav_menus), pembaruan ini dimuat di dua tempat (Seeder dan Migration). Migration baru sengaja ditambahkan agar situs existing mendapatkan perbaikan data fix tanpa perlu me-reset/seed ulang keseluruhan struktur menu yang mungkin sudah di-custom oleh pengguna.

@github-actions

Copy link
Copy Markdown
Contributor

🔄 AI PR Review sedang antri di server...

Proses review akan segera dimulai di background — hasil akan muncul sebagai komentar setelah selesai.
Powered by CrewAI · PR #1658

@habibie11
habibie11 requested a review from vickyrolanda July 13, 2026 10:47
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.

3 participants