Skip to content

Set Primary Key for Notifications Table#259

Merged
maxdml merged 12 commits intodbos-inc:mainfrom
azghr:azghr/pkey
Mar 3, 2026
Merged

Set Primary Key for Notifications Table#259
maxdml merged 12 commits intodbos-inc:mainfrom
azghr:azghr/pkey

Conversation

@azghr
Copy link
Contributor

@azghr azghr commented Feb 28, 2026

No description provided.

@azghr
Copy link
Contributor Author

azghr commented Feb 28, 2026

@maxdml please review this PR.

@maxdml maxdml self-assigned this Mar 2, 2026
@maxdml
Copy link
Collaborator

maxdml commented Mar 3, 2026

Thanks @azghr . We're going to have to wait that #261 is done, so we can add this as migration 10 -- we are maintaining a 1:1 parity on DB migrations across the SDKs.

@maxdml
Copy link
Collaborator

maxdml commented Mar 3, 2026

@azghr I am adding the 9th migration here #262. Can you update this PR to make this the 10th migration? We want to port all the existing migrations to Golang today.

@maxdml maxdml mentioned this pull request Mar 3, 2026
@azghr
Copy link
Contributor Author

azghr commented Mar 3, 2026

Thanks! @maxdml let me handle this

@maxdml maxdml merged commit 55deb65 into dbos-inc:main Mar 3, 2026
4 checks passed
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