feat(ai): migrate Imagen pages to Nano Banana#1846
Conversation
There was a problem hiding this comment.
Code Review
This pull request rebrands the "Imagen" features to "Nano Banana" throughout the application, updating screen names, view models, and model configurations to utilize the GenerativeModel with the gemini-2.5-flash-image model. Key changes include updating copyright headers, navigation titles, and adapting response handling to use inlineDataParts. Review feedback suggests replacing print statements with the existing logger instance for better log management and correcting a typo in the sample data titles.
| print("Image(s) Blocked: \(filteredReason)") | ||
| // 2. Print the reason images were blocked, if any. | ||
| if let blockReason = response.promptFeedback?.blockReason { | ||
| print("Image(s) Blocked: \(blockReason)") |
There was a problem hiding this comment.
Consider using the class's logger instance instead of print for consistency and better log management.
| print("Image(s) Blocked: \(blockReason)") | |
| logger.info("Image(s) Blocked: \(blockReason)") |
| print("Image(s) Blocked: \(filteredReason)") | ||
| // 2. Print the reason images were blocked, if any. | ||
| if let blockReason = response.promptFeedback?.blockReason { | ||
| print("Image(s) Blocked: \(blockReason)") |
There was a problem hiding this comment.
Consider using the class's logger instance instead of print for consistency and better log management.
| print("Image(s) Blocked: \(blockReason)") | |
| logger.info("Image(s) Blocked: \(blockReason)") |
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Per b/498982262,
This PR migrates all of our existing Imagen screens to Nano Banana alternatives. This aligns with the deprecation of the Imagen API, and should offer consumers a prime example of what migration looks like.
Note that for the template, I created a template in the Firebase console that matched the Imagen one. Although, since there's not a default dropdown for a Nano Banana template, I'd be fine with just removing that screen entirely.