Skip to content

[d16-8] Add AutoFill CredentialProvider NSExtensionPoint support (#75) - #76

Merged
rolfbjarne merged 1 commit into
dotnet:d16-8from
rolfbjarne:autofill-credentialprovider-d16-8
Jul 14, 2020
Merged

[d16-8] Add AutoFill CredentialProvider NSExtensionPoint support (#75)#76
rolfbjarne merged 1 commit into
dotnet:d16-8from
rolfbjarne:autofill-credentialprovider-d16-8

Conversation

@rolfbjarne

Copy link
Copy Markdown
Member
  • Add credential provider to ExtensionPoint enumeration

  • Add min version check rule for credential provider ExtensionPoint

  • Add trailing comma to make the next entry's diff smaller

* Add credential provider to ExtensionPoint enumeration

* Add min version check rule for credential provider ExtensionPoint

* Add trailing comma to make the next entry's diff smaller

Co-authored-by: Rolf Bjarne Kvinge <rolf@xamarin.com>
@rolfbjarne rolfbjarne changed the title Add AutoFill CredentialProvider NSExtensionPoint support (#75) [d16-8]Add AutoFill CredentialProvider NSExtensionPoint support (#75) Jul 13, 2020
@rolfbjarne rolfbjarne changed the title [d16-8]Add AutoFill CredentialProvider NSExtensionPoint support (#75) [d16-8] Add AutoFill CredentialProvider NSExtensionPoint support (#75) Jul 13, 2020

@spouliot spouliot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This would get out faster if backported to xcode12 and would result in less chance of having a merge conflict (later).

This would also be automagically backported to d16-8 since Xcode 12 is likely to be release prior to it.

@rolfbjarne

Copy link
Copy Markdown
Member Author

This would get out faster if backported to xcode12 and would result in less chance of having a merge conflict (later).

Good point, I've back ported to xcode12 as well: #39

@rolfbjarne
rolfbjarne merged commit 8208733 into dotnet:d16-8 Jul 14, 2020
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.

3 participants