Skip to content
Snippets Groups Projects

Added migrations table

Merged Felix Delattre requested to merge feature/add-migrations-table into master
All threads resolved!

Migrations table to track changes to the database.

\rfc @nicgar @prehn @marius \approve @ds

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Felix Delattre added 1 commit

    added 1 commit

    Compare with previous version

  • Looks good. I'm fine with merging after considering @ds comments.

  • Felix Delattre added 1 commit

    added 1 commit

    Compare with previous version

  • Author Contributor

    The logic for populating the migrations table can then be perfectly added to the script @prehn is proposing here https://git.gfz-potsdam.de/dynamicexposure/openbuildingmap/database-obmtiles/-/merge_requests/4 - Looks good how this is coming together.

  • Danijel Schorlemmer resolved all threads

    resolved all threads

  • Danijel Schorlemmer approved this merge request

    approved this merge request

  • assigned to @fd

  • Author Contributor

    May I ask for a quick check and feedback from our side, @nicgar and @prehn?

  • Felix Delattre assigned to @prehn and unassigned @fd

    assigned to @prehn and unassigned @fd

  • @nicgar @prehn: Please approve this MR so that we can move on.

  • Karsten Prehn approved this merge request

    approved this merge request

  • Nicolas Garcia Ospina approved this merge request

    approved this merge request

  • Author Contributor

    Please reassign people after approval. If not MRs get lost.

  • Felix Delattre assigned to @fd and unassigned @prehn

    assigned to @fd and unassigned @prehn

  • Felix Delattre added 2 commits

    added 2 commits

    Compare with previous version

  • merged

  • Please register or sign in to reply
    Loading