Skip to content

feat: adds Flyway database migration tooling#104

Open
rahulvikram wants to merge 9 commits into
devfrom
feat/flyway-db-migration
Open

feat: adds Flyway database migration tooling#104
rahulvikram wants to merge 9 commits into
devfrom
feat/flyway-db-migration

Conversation

@rahulvikram

@rahulvikram rahulvikram commented Jun 12, 2026

Copy link
Copy Markdown

This PR:

  • Adds Flyway community database migration tooling to docker-compose startup process
  • Adds notes to documentation about database setup and migration for development
  • Removes old db-init.sql
  • Migration: adds altText to Items and History_Items

Adds notice to documentation about rollbacks not being supported by Flyway community. Instead tells devs to use new migration scripts to fix/reverse changes (which get applied during docker compose startup).
@rahulvikram rahulvikram requested a review from clementsm June 12, 2026 02:46
@rahulvikram rahulvikram self-assigned this Jun 12, 2026
Copilot AI review requested due to automatic review settings June 12, 2026 02:46

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds Flyway Community database migration support to the Docker Compose dev workflow and documents how schema changes should be managed going forward, including an initial migration to add altText columns for accessibility.

Changes:

  • Introduces a flyway Docker Compose service that runs migrations after MariaDB is healthy and before the app starts.
  • Adds developer documentation for migration naming, safety, and running Flyway commands via Docker Compose.
  • Adds V001 migration to add altText columns to Items and History_Items.

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
services/database/migrations/V001__add_altText_to_Items_and_History_Items.sql Adds initial schema migration for altText columns.
services/database/migrations/README.md Documents Flyway migration conventions and usage.
README.md Updates dev DB guidance to Docker Compose + Flyway and fixes env var naming.
DOCKER.md Documents Flyway service behavior and migration commands in Docker workflow.
docker-compose.yml Adds Flyway service and gates app startup on successful migrations.
CONTRIBUTING.md Updates contributor workflow to use the new seed and migration process.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread DOCKER.md Outdated
Comment thread docker-compose.yml Outdated
rahulvikram and others added 2 commits June 11, 2026 19:53
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Switches from using 'latest' to prevent breaking changes from different flyway versions being pulled.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants