Skip to content

APPSINTG-435: AWS DynamoDB Integration - Documentation - #7142

Open
mahendrak-sumo wants to merge 3 commits into
SumoLogic:mainfrom
mahendrak-sumo:APPSINTG-435
Open

mahendrak-sumo wants to merge 3 commits into
SumoLogic:mainfrom
mahendrak-sumo:APPSINTG-435

Conversation

@mahendrak-sumo

@mahendrak-sumo mahendrak-sumo commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator

Purpose of this pull request

Added a new AWS DynamoDB integration including following actions Describe Table and Update Table actions.

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

https://sumologic.atlassian.net/browse/APPSINTG-435

@cla-bot cla-bot Bot added the cla-signed Contributor approved, listed in .clabot file label Sep 16, 2026
@mahendrak-sumo mahendrak-sumo self-assigned this Sep 16, 2026

@amee-sumo amee-sumo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Left inline comments on two gaps, plus a note on the PR description below. Everything else (frontmatter, Actions, External Libraries labeling, Configure section, Required Permissions) matches sibling docs like aws-lambda.md and aws-iam.md, and CI is green.

Separately: the PR description mentions "validation, error-handling, and metadata tests" and "mocked pytest coverage," but this diff only touches the doc file and one screenshot — no test code exists here. This looks copied from the underlying code PR's description rather than written for this docs PR (same pattern I flagged on the Intel 471 PR). Worth rewriting to describe what's actually in this diff.

@amee-sumo amee-sumo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Both flagged items are resolved: "Additional resources" section added correctly, and the version-format question was answered with a deliberate policy explanation (per Sachin and Nitin's request, going forward). PR description now accurately reflects the actual diff. CI is green.

Approving.

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

Labels

cla-signed Contributor approved, listed in .clabot file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants