Skip to content

chore: adding the table getChanges method#87

Merged
kodiakhq[bot] merged 1 commit intomainfrom
chore/add-table-changes
Aug 23, 2023
Merged

chore: adding the table getChanges method#87
kodiakhq[bot] merged 1 commit intomainfrom
chore/add-table-changes

Conversation

@mnorbury
Copy link
Copy Markdown
Collaborator

refs: #85

@mnorbury mnorbury force-pushed the chore/add-table-changes branch from 4aa9b5e to 024661a Compare August 22, 2023 15:48
@mnorbury mnorbury requested a review from erezrokah August 22, 2023 15:48
@mnorbury mnorbury added the automerge Add to automerge PRs once requirements are met label Aug 22, 2023
Copy link
Copy Markdown
Member

@erezrokah erezrokah left a comment

Choose a reason for hiding this comment

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

Looks good. We need this to so plugins can use a shared logic to handle migrations?
I would scope out this logic from the first SDK release and let plugins handle it for now.

@kodiakhq kodiakhq Bot merged commit 4159f14 into main Aug 23, 2023
@kodiakhq kodiakhq Bot deleted the chore/add-table-changes branch August 23, 2023 07:55
@mnorbury
Copy link
Copy Markdown
Collaborator Author

Looks good. We need this to so plugins can use a shared logic to handle migrations? I would scope out this logic from the first SDK release and let plugins handle it for now.

I guess I was on autopilot, it is used by the memdb during migrations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Add to automerge PRs once requirements are met

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants