Skip to content
This repository was archived by the owner on Jul 4, 2025. It is now read-only.

feat: data migration #1692

Merged
merged 11 commits into from
Nov 25, 2024
Merged

feat: data migration #1692

merged 11 commits into from
Nov 25, 2024

Conversation

@vansangpfiev
Copy link
Contributor

@vansangpfiev vansangpfiev commented Nov 15, 2024 •
edited
Loading

Describe Your Changes

  • Create schema_version table to track database/data structure version
  • Add migration logic for version 0

Fixes Issues

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

@vansangpfiev vansangpfiev changed the title feat: data migration [WIP] feat: data migration Nov 18, 2024
@vansangpfiev vansangpfiev marked this pull request as ready for review November 18, 2024 02:02
@vansangpfiev vansangpfiev self-assigned this Nov 18, 2024
@vansangpfiev vansangpfiev changed the title [WIP] feat: data migration feat: data migration Nov 18, 2024
@vansangpfiev vansangpfiev requested review from a team and dan-menlo November 18, 2024 04:57
namchuai
namchuai reviewed Nov 19, 2024
View reviewed changes
namchuai
namchuai approved these changes Nov 19, 2024
View reviewed changes
Copy link
Contributor

@namchuai namchuai left a comment

Choose a reason for hiding this comment

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

✅

@vansangpfiev vansangpfiev merged commit adfbda1 into dev Nov 25, 2024
4 checks passed
@vansangpfiev vansangpfiev deleted the feat/updater-migration branch November 25, 2024 01:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@dan-menlo dan-menlo Awaiting requested review from dan-menlo

1 more reviewer

@namchuai namchuai namchuai approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@vansangpfiev vansangpfiev

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@vansangpfiev @namchuai @sangjanai