Zoymusk opened a new pull request, #120: URL: https://github.com/apache/fineract-loan-origination/pull/120
## What this changes Adds OpenAPI (`@Operation` + `@Tag`) annotations to 4 of the 5 controllers listed as undocumented in #102: `AuthController`, `CustomerLoanApplicationController`, `AdminCustomerController`, and `CustomerProfileController`. Closes #102 partially — see below. ## What's NOT included, and why - `CryptoKeyController` doesn't exist yet — it's introduced by #101 (RSA encryption work), which hasn't merged. Nothing to document there yet. - `AuthController` currently has 2 endpoints (`/login`, `/staff/login`), both documented here. The issue's count of 4 POST endpoints for this file includes 2 encrypted-variant endpoints (`/login/encrypted`, `/staff/login/encrypted`) that are also introduced by #101 and don't exist in the codebase yet. Once #101 merges, I'm happy to open a follow-up PR covering `CryptoKeyController` and the 2 new encrypted endpoints. ## Verification - `mvn compile` — BUILD SUCCESS - Ran the app locally (`docker compose up -d los-db` + `./mvnw spring-boot:run`) and confirmed all added descriptions render correctly in `/swagger-ui/index.html`, matching the target-state screenshot in #102. ## Checklist - [x] Followed the existing `@Operation` + `@Tag` pattern from already-documented controllers (e.g. `LoanApplicationController`) - [x] No implementation code changed — additive only - [x] Verified via running Swagger UI, not just compilation -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
