Skip to content

feat(letter-categories): implement EV5-1-S1 letter categories - #68

Closed
hanchan-dev wants to merge 1 commit into
devfrom
feat/(EV5-1-S1)LetterCategories
Closed

feat(letter-categories): implement EV5-1-S1 letter categories#68
hanchan-dev wants to merge 1 commit into
devfrom
feat/(EV5-1-S1)LetterCategories

Conversation

@hanchan-dev

Copy link
Copy Markdown
Member

Ohh, maksudmu seluruh template PR dari ## Summary sampai Checklist, termasuk bagian Markdown-nya, supaya tinggal copy-paste. Ini full-nya:

## Summary

Implementasi EV5-1-S1 untuk fitur **Letter Categories** sesuai TDD dan API Specification v5.

Migration, repository, endpoint, dan test untuk `letter_categories` telah disiapkan sesuai kontrak yang ditentukan.

## Jira Ticket

Ticket: `EV5-1-S1`

## Type of Change

- [ ] Bug fix
- [x] New feature
- [ ] Refactor
- [ ] Documentation update
- [ ] Configuration change

## Changes Made

- Menambahkan migration dan model `LetterCategory` sesuai struktur `letter_categories` pada TDD.
- Menambahkan `LetterCategoryRepository` dengan method `all()`, `findAllActive()`, dan `findByCode()` sesuai Class Diagram.
- Menambahkan endpoint `GET /letter-categories` sesuai API Specification v5.
- Mendukung 4 kategori: `approval_normal`, `upload_mandiri`, `dokumen_pendukung`, dan `update_data`.
- Menambahkan test untuk repository dan endpoint, termasuk authentication, unique constraint pada `code`, dan boolean cast `is_active`.

## How to Test

1. Checkout branch EV5-1-S1.
2. Pastikan dependency sudah ter-install:
   ```bash
   composer install
   ```
3. Jalankan migration:
   ```bash
   php artisan migrate
   ```
4. Jalankan seluruh test:
   ```bash
   php artisan test
   ```
5. Pastikan test terkait **Letter Categories** berhasil.
6. Test endpoint:
   ```text
   GET /letter-categories
   ```
7. Gunakan authenticated user dan pastikan response sesuai dengan API Specification v5.

## Screenshots (if applicable)

Not applicable — perubahan ini tidak mencakup UI.

## Checklist

- [ ] Kode sudah di-test secara lokal
- [ ] Tidak ada `console.log` / `dd()` / `dump()` yang tertinggal
- [ ] Tidak ada conflict dengan branch target
- [x] Nama branch mengikuti konvensi
- [x] PR ditujukan ke branch `dev` (bukan langsung ke `main`)

@hanchan-dev hanchan-dev closed this Sep 4, 2026
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