Samer-Melhem-FOO opened a new pull request, #6189: URL: https://github.com/apache/fineract/pull/6189
## Description The Client Entity bulk import template did not visually indicate that "Activation Date" is a required column. `ClientEntityImportHandler` requires a valid Activation Date whenever the "Active" column is set to `true`, so imports failed on rows left blank without any indication from the template that the field was mandatory. This changes the column header in `ClientEntityWorkbookPopulator` from `"Activation Date "` to `"Activation Date* "`, consistent with how other conditionally-required columns (e.g. `"Active*"`) are marked on the same sheet. ## JIRA https://issues.apache.org/jira/browse/FINERACT-2715 ## Test plan - [x] `./gradlew :fineract-provider:compileJava` passes - [x] Verified no existing test asserts the previous header string (integration test `ClientEntityWorkbookPopulatorTest` does not hardcode it) -- 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]
