Skip to content

[Dammi] Split off version table writing for migrations#3643

Open
luisa-beerboom wants to merge 14 commits into
OpenSlides:feature/dammifrom
luisa-beerboom:split-off-version-table-writing
Open

[Dammi] Split off version table writing for migrations#3643
luisa-beerboom wants to merge 14 commits into
OpenSlides:feature/dammifrom
luisa-beerboom:split-off-version-table-writing

Conversation

@luisa-beerboom

@luisa-beerboom luisa-beerboom commented Jun 29, 2026

Copy link
Copy Markdown
Member

TODO:

  • See if all those self.cursor.connection.commit() are really necessary
  • Write test to see what happens if a mig fails (is migration_failed written?)
  • remove **kwargs from data_manipulation
  • linting

@luisa-beerboom luisa-beerboom self-assigned this Jun 29, 2026
@luisa-beerboom luisa-beerboom added the migration Introduces a new migration label Jun 29, 2026
@luisa-beerboom luisa-beerboom force-pushed the split-off-version-table-writing branch from bbbdc65 to 677733a Compare June 29, 2026 12:27
@luisa-beerboom luisa-beerboom changed the base branch from main to feature/dammi June 29, 2026 12:27
@luisa-beerboom luisa-beerboom requested a review from hjanott July 13, 2026 16:03
@luisa-beerboom luisa-beerboom marked this pull request as ready for review July 13, 2026 16:03
Comment thread openslides_backend/migrations/migration_handler.py Outdated
Comment thread openslides_backend/migrations/migration_manager.py Outdated
Comment thread openslides_backend/migrations/migration_handler.py Outdated
Comment thread openslides_backend/migrations/migration_handler.py Outdated
Comment thread openslides_backend/migrations/migration_handler.py Outdated
Comment thread tests/system/migrations/test_0100_init_reldb.py Outdated
Comment thread tests/system/migrations/test_0100_init_reldb.py Outdated
Comment thread tests/system/migrations/test_0100_init_reldb.py Outdated
Comment thread openslides_backend/migrations/migration_manager.py Outdated
Comment thread openslides_backend/migrations/migration_manager.py Outdated
@hjanott hjanott assigned luisa-beerboom and unassigned hjanott Jul 14, 2026
@luisa-beerboom luisa-beerboom requested a review from hjanott July 15, 2026 11:08
@luisa-beerboom luisa-beerboom removed their assignment Jul 15, 2026
Comment thread tests/system/migrations/test_0100_init_reldb.py
@hjanott hjanott assigned luisa-beerboom and unassigned hjanott Jul 15, 2026

@hjanott hjanott left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yay! Good work!

@luisa-beerboom luisa-beerboom enabled auto-merge (squash) July 16, 2026 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

migration Introduces a new migration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants