LE0-Lin opened a new pull request, #481: URL: https://github.com/apache/fineract-backoffice-ui/pull/481
## What and why Interest pauses could be created and deleted, but a mistyped date could not be corrected without removing the pause and creating a second record. This adds an edit action and guarded route, reuses the existing form in edit mode, pre-fills the selected variation, and saves it through the generated PUT operation. The edit loader accepts both ISO dates declared by the generated OpenAPI model and the `[year, month, day]` shape returned by older Fineract deployments. Delete failures are now left to the global error interceptor instead of being hidden in the console. Closes #283. ## Verification - Focused Vitest: 10/10 passed (list navigation, create POST, edit pre-fill, edit PUT, rejected update) - Mocked Playwright: `loan-servicing-gaps.spec.ts` 14/14 passed with one worker, including list -> edit -> save and an exact PUT URL/body assertion - `npm run lint` - `npm run i18n:check` - `npm run check:icons` - `npm run typecheck:e2e` - Production build passed (with the existing header component budget warning) - Prettier check passed for all seven changed files The complete unit suite currently reports 1,343 passed and 37 failures in the unrelated loan-product and fixed-deposit-product accounting specs. I reproduced the same 37 failures after stashing every change in this PR and running against clean `upstream/main`. A real-backend e2e was not run locally because Docker is unavailable on this machine. The PUT method, path, parameter order, and request shape were cross-checked against `LoanInterestPauseApiResource`, `InterestPauseRequestDto`, and Fineract's `Verify interest pause update` acceptance scenario. ## Screenshots Not attached. The UI change is the standard `create-outline` row action and the existing form with an edit title; the complete workflow is exercised by the mocked Playwright test above. ## Checklist - [x] I did not hand-edit generated files under `src/app/api/`. - [x] No new browser globals or imperative third-party APIs were introduced; the adapter-boundary item is not applicable. - [x] User-facing strings use translation keys. - [x] I added focused unit coverage for both create and edit modes. - [x] I added mocked e2e coverage for the complete edit workflow; real-backend execution is documented above. - [x] Commits are signed. -- 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]
