Skip to content

fix(ai): fix compile error due to session resumption#1852

Open
daymxn wants to merge 4 commits intomainfrom
dlr/fix-match-for-session-resumption
Open

fix(ai): fix compile error due to session resumption#1852
daymxn wants to merge 4 commits intomainfrom
dlr/fix-match-for-session-resumption

Conversation

@daymxn
Copy link
Copy Markdown
Member

@daymxn daymxn commented Apr 28, 2026

Per b/507503421,

This PR updates the LiveViewModel match case on the response payload to also have a case for . sessionResumptionUpdate (which was added with firebase-ios-sdk#15904), to fix the compile errors caused from a non exhaustive switch statement.

This PR also bumps the min version to align with the 12.13.0 release, so it can properly take advantage of this change.

@daymxn daymxn requested a review from ncooke3 April 28, 2026 19:30
@daymxn daymxn self-assigned this Apr 28, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Firebase iOS SDK dependency to version 12.13.0 and adds a handler for the sessionResumptionUpdate case in LiveViewModel. The reviewer correctly identified that the new enum case includes associated values, which requires a pattern match in the switch statement to avoid a compilation error.

daymxn and others added 2 commits April 28, 2026 14:43
…Model.swift

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
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.

2 participants