Skip to content

Update ADK doc according to issue #1350 - 6#1361

Closed
adk-bot wants to merge 4 commits into
mainfrom
agent-changes-20260227-003609
Closed

Update ADK doc according to issue #1350 - 6#1361
adk-bot wants to merge 4 commits into
mainfrom
agent-changes-20260227-003609

Conversation

@adk-bot

@adk-bot adk-bot commented Feb 27, 2026

Copy link
Copy Markdown
Collaborator

This pull request adds documentation for the new add_memory method, which allows for direct memory ingestion. It also explains how VertexAiMemoryBankService supports this via memories.create or memories.generate (with consolidation) based on configuration.

@netlify

netlify Bot commented Feb 27, 2026

Copy link
Copy Markdown

Deploy Preview for adk-docs-preview ready!

Name Link
🔨 Latest commit 88f0060
🔍 Latest deploy log https://app.netlify.com/projects/adk-docs-preview/deploys/6a4eb5244def030008f2d3f8
😎 Deploy Preview https://deploy-preview-1361--adk-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@zyantw zyantw self-assigned this Jun 18, 2026
@koverholt

Copy link
Copy Markdown
Collaborator

@zyantw, this PR has a merge conflict with main and needs a rebase with content fixes/edits before I can review. I confirmed it's a valid conflict to resolve (not a moved/deleted page). I'll re-review once it's rebased and free of conflicts.

zyantw added 2 commits July 8, 2026 13:56
Simplified dense sections, corrected markdown formatting errors, and fixed typos to align with documentation standards. Corrected some sections from the original page and the agent's PR that were a little clunky. Erased sentences inside paragraph's ,Joe's request, and included commas instead. All this while including the necessary update.
@zyantw zyantw requested a review from koverholt July 8, 2026 20:38
@zyantw

zyantw commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Rendered page: https://deploy-preview-1361--adk-docs-preview.netlify.app/sessions/memory/
Agent's PR: #1358
Original issue: #1350

@koverholt koverholt 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.

Thanks for the proposed edits. The biggest issue is that the main deliverable for #1350 item 6 is missing. So we can pull in the desired content from PR #1358 into here. The rest of the comments and suggestions are in-line. Will re-review when you are ready.

Comment thread docs/sessions/memory.md
Comment thread docs/sessions/memory.md
Comment thread docs/sessions/memory.md
Comment thread docs/sessions/memory.md
Comment thread docs/sessions/memory.md
Comment thread docs/sessions/memory.md
Comment thread docs/sessions/memory.md
Comment thread docs/sessions/memory.md
Comment thread docs/sessions/memory.md
@zyantw

zyantw commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Closed with: #1951

@zyantw zyantw closed this Jul 14, 2026
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.

4 participants