Skip to content

Conversation

@pcholakov
Copy link
Collaborator

No description provided.

Copy link

@muhamadazmy muhamadazmy left a comment

Choose a reason for hiding this comment

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

Thank you @pcholakov for guiding me through this and doing all the work 😅
The change looks good to me. The PR also a good reference to go back to if I need to add make more changes.

@pcholakov pcholakov force-pushed the pavel/qvzslonrxzxx branch 5 times, most recently from 516d575 to 3626a25 Compare November 6, 2025 13:56
@pcholakov pcholakov changed the base branch from pavel/rrvlvkxxuopq to main November 6, 2025 14:07
Copy link

@muhamadazmy muhamadazmy left a comment

Choose a reason for hiding this comment

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

I left one comment about the ddb table key name

} else {
table = new ddb.Table(stack, "DynamoDbMetadataTable", {
partitionKey: {
name: "kind",

Choose a reason for hiding this comment

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

I have changed the partition key name to pk instead of kind (key was a reserved word and did not work)

Suggested change
name: "kind",
name: "pk",

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.

3 participants