AshharAhmadKhan commented on PR #5491: URL: https://github.com/apache/fineract/pull/5491#issuecomment-3909774130
@IOhacker Updated as requested! Changes made: ✅ Removed sample_data SQL files (barebones_db.sql and load_sample_data.sql) ✅ Kept only Liquibase migration approach ✅ Single commit with GPG signature ✅ Spotless applied successfully ✅ Used explicit report names in migration (26 permissions + 44 reports) The migration now handles: - Child table deletion first (stretchy_report_parameter) - Explicit listing of all 26 Pentaho permission codes - Explicit listing of all 44 Pentaho report names - PreConditions for idempotent execution - Cross-database compatibility with LOWER() functions Ready for 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]
