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

Changes to use the library in .Net 7 winforms app #197

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

statler
Copy link

@statler statler commented Mar 29, 2023

Made some changes to get your (really useful) auth working for me in a .Net 7 winforms app.

Changes:

  • made some classes more accessible to hook in from a winform - didn't have access to necessary methods.
  • A few library updates, but they weren't essential.
  • adds the true to the project file in WPF. This shouldn't affect WPF at all
  • change parameter type to WebAuthenticationBroker AuthenticateAsync to support management by a winform as well as WPF

It would be great if you could incorporate this into your published library. Happy to discuss.

@bezysoftware
Copy link
Collaborator

Frankly I would prefer if it were native winforms app, with it's own native UI. How is this interoperability with WPF controls done?

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.

2 participants