Skip to content

Handle null values for FlutterStore.flutterClientId.#9820

Merged
auto-submit[bot] merged 3 commits intoflutter:masterfrom
kenzieschmoll:flutter-store
May 5, 2026
Merged

Handle null values for FlutterStore.flutterClientId.#9820
auto-submit[bot] merged 3 commits intoflutter:masterfrom
kenzieschmoll:flutter-store

Conversation

@kenzieschmoll
Copy link
Copy Markdown
Member

Required bug fix for the devtools release: #9819. This also bumps the flutter version to the latest.

@kenzieschmoll kenzieschmoll requested review from a team and bkonyi as code owners May 5, 2026 18:28
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 devtools_shared to version 13.0.1 to handle null values for FlutterStore.flutterClientId. The changes involve making the flutterClientId getter nullable and adding null-coalescing logic in the server API. Feedback indicates that changing the return type to String? constitutes a breaking change that should be avoided in a patch release; instead, the getter should return an empty string for null values. Additionally, the reviewer noted that the implementation should verify if analytics are enabled before returning the client ID to comply with existing policy.

Comment thread packages/devtools_shared/lib/src/server/flutter_store.dart Outdated
Comment thread packages/devtools_shared/lib/src/server/server_api.dart Outdated
@kenzieschmoll kenzieschmoll added the autosubmit Merge PR when tree becomes green via auto submit App label May 5, 2026
@auto-submit auto-submit Bot merged commit 23a1bf0 into flutter:master May 5, 2026
89 of 90 checks passed
@kenzieschmoll kenzieschmoll deleted the flutter-store branch May 5, 2026 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App release-notes-not-required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants