-
Notifications
You must be signed in to change notification settings - Fork 9.8k
[shared_preferences] Add shared_preferences_windows #2988
[shared_preferences] Add shared_preferences_windows #2988
Conversation
All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the ℹ️ Googlers: Go here for more info. |
8f5b134
to
351244a
Compare
…the main plugin's tests
…ath_provider_windows
This is now working, and passing the experimental CI bots. However, looking again at the way it's doing path storage, and comparing with the Linux implementation, it makes a lot more sense to hold off on this until path_provider_windows is landed, and then use it to get the storage location instead of having the Win32 code duplicated here (with different logic for constructing the subpath, as compared to path_provider). Once path_provider_windows is in I'll rework this to depend on it, as we do with shared_preferences_linux. |
@franciscojma86 Can you approve the co-authorship here when you get a chance? I've done a fair amount of rewriting based on what we learned from some of the Linux plugins, especially aligning it with the basic structure of shared_preferences_linux due the fact that path_provider_windows is now mostly complete, but it's still very much based on your PR. |
@googlebot I consent. |
CLAs look good, thanks! ℹ️ Googlers: Go here for more info. |
So exciting! |
This is essentially done, it just needs a trivial pubspec updated after path_provider_windows 0.0.2 is published (#2818). |
packages/shared_preferences/shared_preferences_windows/lib/shared_preferences_windows.dart
Show resolved
Hide resolved
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 don't think we're calculating the path where the shared preferences are stored correctly.
...ages/shared_preferences/shared_preferences_windows/test/shared_preferences_windows_test.dart
Show resolved
Hide resolved
Windows implementation of the shared_preferences plugin, using Dart+FFI.
Windows implementation of the shared_preferences plugin, using Dart+FFI.
Windows implementation of the shared_preferences plugin, using Dart+FFI.
Description
Dart implementation of the shared_preferences plugin using ffi for Windows.
This is #2631, trimmed down to the initial stand-alone PR that adds shared_preferences_windows, since it must be landed and published before the shared_preferences registration and endorsement can land.
Related Issues
flutter/flutter#41719
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]
). This will ensure a smooth and quick review process.///
).flutter analyze
) does not report any problems on my PR.Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?