-
Notifications
You must be signed in to change notification settings - Fork 3.6k
[local_auth_darwin] macOS Support #6267
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
134 commits
Select commit
Hold shift + click to select a range
532ed7f
Beginning of macos support
alexrabin-sentracam 0eb64ee
Merge branch 'flutter:main' into macos-support
alexrabin-sentracam 25ca34e
Made macos deployment target 10.15
alexrabin-sentracam c57ac02
Merge branch 'macos-support' of https://github.com/alexrabin-sentraca…
alexrabin-sentracam 00bd678
Separate ios and macos code
alexrabin-sentracam 509708f
Adjusted format
alexrabin-sentracam fd6c539
Updated change log
alexrabin-sentracam 44dd8c8
Removed duplicated analysis_options.yaml
4f68b52
Removed unnecessary widget_test in darwin
8646867
Added license blocks in macos example files
676b0eb
Changed iOSAuthMessages to DarwinAuthMessages
bcee2e7
Merge branch 'main' into macos-support
alexrabin-sentracam 4aab9d8
Added macos comment
1bded66
Changed macOS capitalization
7335837
Removed unnecessary test, added comment, and changed version number
849c07b
Removed prettier ignore
c810eb4
Reset upgrade version
6867b3a
Revert "Changed iOSAuthMessages to DarwinAuthMessages" and add MacOSA…
995f41d
Changed podspec to use ios deployment_target
8ab5d09
Checks if macos is 10.15 or higher before checking for faceid
507470e
Shows correct window for registrar
05b52ae
Changed wording
8f35101
Merge branch 'main' into macos-support
alexrabin-sentracam 2b0e6d6
Combined @available statements and renamed keywindow
c808c82
Created initWithRegistrar initializer
64bd0b7
Moved runner tests
7dab5b4
Merge branch 'main' into macos-support
alexrabin-sentracam 7ff9871
Fixed format issues
04f6a76
Revert "Moved runner tests"
04c8f12
Fixed tests and platform available check
64366dd
Merge branch 'main' into macos-support
alexrabin-sentracam 7babc9c
Merge branch 'main' into macos-support
alexrabin-sentracam b71f172
Merge branch 'main' into macos-support
alexrabin-sentracam cfabac0
Merge branch 'main' into macos-support
alexrabin-sentracam 5bc080e
Added macos tests
d794eb9
Fixed format
74ae648
Added info.plist key and added license heading
2374375
Added xml language to readme
f053fce
Upgraded project settings
04049a3
Reset example macos project
9be16c3
Fixed macos tests
ff533bc
Removed settings button on macos due to no native support from Apple …
783b0cc
Added @available check
5e9b627
Merge branch 'main' into macos-support
alexrabin-sentracam 1748039
Merge branch 'main' into macos-support
alexrabin-sentracam ebdb5f1
Merge branch 'main' into macos-support
alexrabin-sentracam b73254a
Merge branch 'main' into macos-support
alexrabin-sentracam c4c9f3b
Merge branch 'main' into macos-support
alexrabin-sentracam 6ca1476
Merge branch 'main' into macos-support
alexrabin-sentracam 69db0ec
Merge branch 'main' into macos-support
alexrabin-sentracam a39e82a
Merge branch 'main' into macos-support
alexrabin-sentracam 59ce98a
Merge branch 'main' into macos-support
alexrabin-sentracam e54c7ea
Merge branch 'main' into macos-support
alexrabin-sentracam b8ca53a
Merge branch 'main' into macos-support
alexrabin-sentracam 018fe8f
Merge branch 'main' into macos-support
alexrabin-sentracam ff38a26
Merge branch 'main' into macos-support
alexrabin-sentracam e76f6bc
Addressed additional feedback
72b8212
Fixed plugin tests
113195e
Merge branch 'main' into macos-support
alexrabin-sentracam 7525f01
Merge branch 'main' into macos-support
alexrabin-sentracam 6330a9e
Merge branch 'main' into macos-support
alexrabin-sentracam 79ebae9
Made initWithRegistrar the designated initializer
0c84f2b
Merge branch 'macos-support' of https://github.com/alexrabin-sentraca…
c84a376
Merge branch 'main' into macos-support
alexrabin-sentracam 5dc7c75
Merge branch 'main' into macos-support
alexrabin-sentracam c33ed13
Merge branch 'main' into macos-support
alexrabin-sentracam 186d89c
Updated messaging
f9e11db
Merge branch 'main' into macos-support
alexrabin-sentracam c45084b
Remove gotosettingsbutton on macos and didn't make it required
7003142
Merge branch 'main' into macos-support
alexrabin-sentracam 18453dc
Merge branch 'main' into macos-support
alexrabin-sentracam fc14107
Merge branch 'main' into macos-support
alexrabin-sentracam 5840a73
Merge branch 'flutter:main' into macos-support
alexrabin-sentracam ce1fa37
Merge branch 'main' into macos-support
alexrabin-sentracam ad83e24
Adjusted format
8beb6dd
Merge branch 'main' into macos-support
alexrabin-sentracam 067a0c0
Added initWithContextFactoryAndRegistrar since tests failing due to m…
e3e89d4
Fixed mock test error
a9167cc
Merge branch 'main' into macos-support
alexrabin-sentracam 0f56ad6
Created base StubAlertFactory
17bd1c7
Merge branch 'macos-support' of https://github.com/alexrabin-sentraca…
2441440
Added test to check if the alert window is using the registrar window
8af7ec7
Added overridemacosmessages
4fcbc5d
Removes default false for overrideUseMacOSAuthMessages
8bd0531
Merge branch 'main' into macos-support
alexrabin-sentracam 44e5ae3
Merge branch 'main' into macos-support
alexrabin-sentracam 51b3b02
Merge branch 'main' into macos-support
alexrabin-sentracam 6d0684f
Merge branch 'main' into macos-support
alexrabin-sentracam ca5789f
Merge branch 'main' into macos-support
alexrabin-sentracam 6d5a0c6
Merge branch 'main' into macos-support
alexrabin-sentracam 11e6991
Merge branch 'main' into macos-support
alexrabin-sentracam 316bb68
Updated method names
3728490
Fixed comment format
5b72aea
Fixed macos tests
6dd9092
Merge branch 'main' into macos-support
alexrabin-sentracam b5ea028
Merge branch 'main' into macos-support
alexrabin-sentracam dd545e5
Merge branch 'main' of https://github.com/flutter/packages into macos…
85678af
Upgraded version to 1.4.0
679854a
Removed copy property
464ae48
Checks parent window correctly and uses nsa;ert object
05bc529
Merge branch 'main' into macos-support
alexrabin-sentracam d3ce499
Merge branch 'main' into macos-support
alexrabin-sentracam 57c50d3
Added nonnulll keyword
860d198
Updated format
9224412
Added nonnull
9c345f9
Added non null to create alert
71a2824
Merge branch 'main' into macos-support
alexrabin-sentracam 0d420c5
Updated elif check
3415984
Merge branch 'macos-support' of https://github.com/alexrabin-sentraca…
09236d6
Added target conditionals import
b0de86c
Added declaration comments
288b04b
Merge branch 'main' into macos-support
alexrabin-sentracam 23305d0
Merge remote-tracking branch 'upstream/main' into macos-support
2293ff6
Merge branch 'main' into macos-support
alexrabin-sentracam 46ce252
Merge branch 'main' into macos-support
alexrabin-sentracam 0f7dac8
Merge branch 'main' into macos-support
alexrabin-sentracam a91e4e9
Fixed swift tests
e5c9b97
Merge branch 'main' into macos-support
alexrabin-sentracam f90387f
Merge branch 'main' into macos-support
alexrabin-sentracam 9024825
Merge branch 'main' into macos-support
alexrabin-sentracam 91a6d10
Merge branch 'main' into macos-support
alexrabin-sentracam 2b2ebab
Merge branch 'main' into macos-support
stuartmorgan-g 8c9d1d1
License fix
stuartmorgan-g ee97d66
Group factories in interface
stuartmorgan-g 211bc52
Update tests and implementation to avoid the need for OCMock.
stuartmorgan-g fabc0d4
Fix alert wrapper creation; tested manually
stuartmorgan-g 568c0ea
Remove OCMock
stuartmorgan-g 371124c
fixup: remove bridging header too
stuartmorgan-g 3b7b5b9
Finish removing OCMock from tests using a view provider protocol, and…
stuartmorgan-g a6a0019
Minor diff cleanup
stuartmorgan-g 25b6411
Re-add a removed comment, fix naming of macOS default
stuartmorgan-g 5e9af66
Fix constant name in unit test
stuartmorgan a8a4317
Adjust the ifdef and comment in the test
stuartmorgan-g c3e81e3
Clean up deployment targets
stuartmorgan-g File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Added initWithContextFactoryAndRegistrar since tests failing due to m…
…issing context factory
- Loading branch information
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -17,6 +17,7 @@ | |
|
|
||
| @interface FLALocalAuthPlugin () | ||
| /// Returns an instance that uses the given factory to create LAContexts. | ||
|
||
| - (instancetype)initWithRegistrar:(NSObject<FlutterPluginRegistrar> *)registrar | ||
| - (instancetype)initWithContextFactoryAndRegistrar:(NSObject<FLADAuthContextFactory> *)factory | ||
| andRegistrar:(NSObject<FlutterPluginRegistrar> *)registrar | ||
| NS_DESIGNATED_INITIALIZER; | ||
| @end | ||
alexrabin-sentracam marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Declaration comments only go on the declaration, they should not be copied to the definition.