New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add migration to support SQS #12676
Add migration to support SQS #12676
Conversation
Plan Error
|
This cannot be merged until the migration system is in place |
@@ -17,6 +17,7 @@ import { LinkDocument } from "@budibase/types" | |||
class LinkDocumentImpl implements LinkDocument { | |||
_id: string | |||
type: string | |||
tableId: string |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm a little uncertain about this name, because it's not actually a table ID. Could be confusing when reading the code to later find out that it's 2 conjoined table IDs.
Perhaps lookupId
?
_rev?: string | undefined | ||
createdAt?: string | number | undefined | ||
updatedAt?: string | undefined |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are these necessary? LinkDocument
inherits from Document
, which defines these already.
Description
Adding a new app migration to support the future SQS implementation. This will require a deterministic field to support joins, so we are adding a tableId field to internal links in order to achieve this