Skip to content

Phase 1: ADK Onboarding journey revision #739

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Oct 9, 2025
Merged

Phase 1: ADK Onboarding journey revision #739

merged 5 commits into from
Oct 9, 2025

Conversation

joefernandez
Copy link
Collaborator

  • Quickstart: split by language and reduce time
  • Build your agent: new section for agent building
  • Home page: Simplify onboarding choices
  • Surface "About ADK" in navigation

@joefernandez
Phase 1: ADK Onboarding journey revision
6c19f2f 
- Quickstart: split by language and reduce time
- Build your agent: new section for agent building
- Home page: Simplify onboarding choices
- Surface "About ADK" in navigation
koverholt
koverholt requested changes Oct 6, 2025
View reviewed changes
Copy link
Collaborator

@koverholt koverholt left a comment

Choose a reason for hiding this comment

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

Thanks for the revisions! I listed a few optimizations for the Python page, and the Java quickstart page needs some additional reworking. Overall I love the more streamlined getting started pages and the reorganization of the "Build your agent" tutorials!

@joefernandez
Updates from first review round and Java research
7228cb1 
- completely revised Java quickstart
koverholt
koverholt requested changes Oct 8, 2025
View reviewed changes
Copy link
Collaborator

@koverholt koverholt left a comment

Choose a reason for hiding this comment

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

Thanks for the updates, the updated Java quickstart page is looking great! The main blocking issue that I ran into is on the Java quickstart page when running with the web UI, and I left a few other minor comments.

koverholt
koverholt reviewed Oct 9, 2025
View reviewed changes
Copy link
Collaborator

@koverholt koverholt left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Thanks for the additional edits, this is looking great!

Final review notes:

  1. Remove export from Python quickstart command related to .env
  2. Switch order of Windows / Mac-Linux tabs with code blocks / commands
  3. Update pom.xml versions / snippet on advanced install page

koverholt
koverholt approved these changes Oct 9, 2025
View reviewed changes
Copy link
Collaborator

@koverholt koverholt left a comment

Choose a reason for hiding this comment

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

Looks great, thanks for the improvements!

@joefernandez joefernandez merged commit b947bc8 into google:main Oct 9, 2025
2 of 3 checks passed
@koverholt koverholt mentioned this pull request Oct 10, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@koverholt koverholt koverholt approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@joefernandez @koverholt