Skip to content
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

[shared_preferences] update List<String> encode/decode and other simple breaking changes #8335

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

tarrinneal
Copy link
Contributor

@tarrinneal tarrinneal commented Dec 20, 2024

Follow up to #8187

Updates Android List<String> encoding/decoding to use JSON to avoid potential future breakages from restricted list types.

fixes flutter/flutter#124420 [bigint and set read support on android removal]

Moves type errors to argument errors on foundation and android packages

@tarrinneal tarrinneal marked this pull request as ready for review January 3, 2025 12:25
@tarrinneal tarrinneal requested a review from reidbaker as a code owner January 3, 2025 12:25
@tarrinneal
Copy link
Contributor Author

@stuartmorgan ready for review here as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[shared_preferences] Remove BigInt and Set<String> support from Android implementation
1 participant