Skip to content

Refactor/reorganize api routes - #72

Merged
hanchan-dev merged 2 commits into
devfrom
refactor/reorganize-api-routes
Sep 4, 2026
Merged

Refactor/reorganize api routes#72
hanchan-dev merged 2 commits into
devfrom
refactor/reorganize-api-routes

Conversation

@hanchan-dev

Copy link
Copy Markdown
Member

Summary

Merapikan dan mengelompokkan ulang route API di routes/api.php berdasarkan prefix dan fitur (approval flows, citizens, letters, notifications, regions, users, dst), lalu diurutkan alfabetis per kelompok. Tidak ada perubahan pada path, logic, atau behavior endpoint — murni reorganisasi untuk memudahkan maintenance ke depannya.

Jira Ticket

Ticket:

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Documentation update
  • Configuration change

Changes Made

  • Mengelompokkan route dengan prefix yang sama (hamlets, rws, rts, rt, rw, kasi, kadus, notifications, citizens, users, letters) menggunakan Route::prefix()->group()
  • Mengurutkan kelompok route secara alfabetis, dengan route auth (logout) diletakkan paling atas
  • Memisahkan route approval rt/rw dari route data region rts/rws agar tidak tertukar
  • Merapikan komentar OfficialController yang masih di-nonaktifkan
  • Tidak ada perubahan pada path, parameter, middleware, atau controller logic

How to Test

  1. Checkout branch ini dan jalankan php artisan route:list untuk membandingkan daftar route dengan branch sebelumnya (pastikan jumlah dan path route sama persis)
  2. Jalankan test suite (jika ada) untuk memastikan semua endpoint masih berfungsi seperti sebelumnya
  3. Coba beberapa endpoint kunci secara manual (misal /api/letters, /api/rt/letters, /api/users) untuk memastikan response tidak berubah

Screenshots (if applicable)

Tidak ada perubahan UI.

Checklist

  • Kode sudah di-test secara lokal
  • Tidak ada console.log / dd() / dump() yang tertinggal
  • Tidak ada conflict dengan branch target
  • Nama branch mengikuti konvensi (feature/nama-fitur, fix/nama-bug)
  • PR ditujukan ke branch dev (bukan langsung ke main)

@hanchan-dev
hanchan-dev merged commit 5c3499c into dev Sep 4, 2026
3 checks passed
@hanchan-dev
hanchan-dev deleted the refactor/reorganize-api-routes branch September 4, 2026 09:36
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.

1 participant