-
Notifications
You must be signed in to change notification settings - Fork 11
Update Agentic App Manager for Google I/O 25 #24
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
Conversation
@@ -1,5 +1,5 @@ | |||
# Uncomment this line to define a global platform for your project | |||
# platform :ios, '12.0' | |||
platform :ios, '13.0' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You'll need to update macos/Podfile
as well. Or, just delete the macos
Runner.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! I meant to delete it. Thank ya!
body: const MyMailScreen(), | ||
); | ||
} | ||
// Run this app instead for realtime audio streaming: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd suggest having two main.dart
instances, one for the default impl, and the other one (say audio_streaming_main.dart
) for audio streaming.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't know that was an option! Done!
I'd suggest changing the top level directory, while you are at it. |
All your suggestions have been implemented @domesticmouse! Thank you!! Waiting for the new release to drop and I'll be ready to merge :) |
@domesticmouse Looks like Colorist is breaking the tests because it's using |
TODO on Tues May 20:
Vertex AI in Firebase ReleaseFirebase AI Logic