Skip to content
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

Feat/batch get #656

Draft
wants to merge 409 commits into
base: v1
Choose a base branch
from
Draft

Conversation

guillaumeduboc
Copy link

No description provided.

ThomasAribart and others added 24 commits January 2, 2024 00:35
…ansform-option

Test and improve transform option
…cloning-step-as-well

use generators for cloning step as well
…eric-type

[v1] Chore : improve type for build methods
…property-on-warm-schemas

add and use freeze property on warm schemas
…n-detecting-invalid-item

print pk and sk when detecting an invalid saved item
…erties-with-symbols

hide operation properties with symbols
…ing-object-timestamp-values

fix bug when providing incomplete object timestamp values
@ThomasAribart ThomasAribart self-assigned this Jan 26, 2024

if (commands.length === 0) {
throw new DynamoDBToolboxError('operations.incompleteCommand', {
message: 'BatchGet command incomplete: No items supplied'
Copy link

Choose a reason for hiding this comment

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

Suggested change
message: 'BatchGet command incomplete: No items supplied'
message: 'BatchGet command incomplete: No item supplied'

import { BatchGetOptions } from './options'

export const buildBatchGetCommandInput = (
commands: GetBatchItemRequest[],
Copy link

Choose a reason for hiding this comment

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

Suggested change
commands: GetBatchItemRequest[],
requests: GetBatchItemRequest[],

const TableProjections: { [key: string]: any } = {}

if (commands.length === 0) {
throw new DynamoDBToolboxError('operations.incompleteCommand', {
Copy link

Choose a reason for hiding this comment

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

Suggested change
throw new DynamoDBToolboxError('operations.incompleteCommand', {
throw new DynamoDBToolboxError('operations.incompleteOperation', {

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants