AshharAhmadKhan commented on PR #5491: URL: https://github.com/apache/fineract/pull/5491#issuecomment-3903321226
@IOhacker Thank you for the feedback! I've updated the PR to include a Liquibase migration file as requested. Changes made: - ✅ Added migration file: `0210_remove_pentaho_reports.xml` - ✅ Included in `changelog-tenant.xml` - ✅ Deletes 26 Pentaho permission entries from `m_permission` table - ✅ Deletes 44 Pentaho report entries from `stretchy_report` table - ✅ Kept the sample data changes for new installations The Liquibase migration ensures existing installations are cleaned up, while the sample data changes prevent new installations from including these legacy Pentaho entries. Ready for re-review! -- 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]
