-
Notifications
You must be signed in to change notification settings - Fork 59
docs(WebAuth): Add Information about the full list of available to be used Properties in appsettings and add some usefull docs and sample links #2884
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
base: main
Are you sure you want to change the base?
Conversation
|
|
|
|
|
Added more links to the MS and Dotnet docs and Samples I found usefull |
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.
Pull Request Overview
This PR adds comprehensive documentation about authentication configuration options and references to the Uno Extensions Authentication documentation. The primary purpose is to provide developers with a complete reference for available WebAuth configuration properties and additional learning resources.
Key changes:
- Documents all available WebConfiguration properties for the WebAuthenticationProvider with JSON configuration examples
- Adds extensive reference links to Microsoft authentication documentation and alternative libraries
- Includes information about the IAuthenticationTokenProvider interface in the HTTP overview
Reviewed Changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| doc/Learn/Http/HttpOverview.md | Adds IAuthenticationTokenProvider documentation and additional HTTP reference links |
| doc/Learn/Authentication/AuthenticationOverview.md | Adds comprehensive WebConfiguration options documentation and extensive authentication reference links |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
|
|
|
|
|
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.
Pull Request Overview
Copilot reviewed 2 out of 3 changed files in this pull request and generated 5 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
|
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.
Pull Request Overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
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.
Pull Request Overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
@jeromelaban the markdown validation seems now to be the only one failing, but I dont know how I can fix this. Having this problem in the last time almost every PR but the Issue on the VS Developer Community to have them fix this in Visual Studio keeps getting closed even when I respond them with their requested informations... Can you maybe tell, what I can do to have the line endings fixed in my PR's? the editorconfig seems also to always make the tabs partially set to spaces if I change code file lines and sets the line endings to "mixed" "lf" instead of "CRLF" (.cs) and in markdown its "just" the line endings... the EOF and editorconfig is meanwile issued on multiple Uno repos (studio, templates and I think the extensions one too) but that it now makes the CI/CD fail for allmost all PR's... and I think the "closes issue" tag is not working for multiple Issues. How do I have to set it so its closing all of them on merge? |
|
|
… in appsettings chore: fix typo Co-authored-by: Copilot <[email protected]> chore: fix typo Co-authored-by: Copilot <[email protected]>
chore: fix typo Co-authored-by: Copilot <[email protected]> chore: fix typo Co-authored-by: Copilot <[email protected]> chore: applying PR rewording suggestion Co-authored-by: Copilot <[email protected]>
…guration, do's and dont do's
2d6db34 to
844b0fc
Compare
|
|
GitHub Issue (If applicable): closes #2869 ,closes #2868 , closes #2865
PR Type
What kind of change does this PR introduce?
What is the current behavior?
The How-To and Overview are only mentioning the StartLogin Path and the one for Logout, but as there are lots more available and possibly relevant to know about, this
What is the new behavior?
There are lots more Properties available and possibly relevant to know about and which are meant to be added by this PR
PR Checklist
Please check if your PR fulfills the following requirements:
Screenshots Compare Test Runresults.Other information
Trying to figure out until now, how the
PrepareLoginUriand other Functions are meant to be used, which would also good to know about, how to set this up. If somone knows about this, please let me know!Internal Issue (If applicable):