Skip to content

Conversation

@vatsashah45
Copy link
Contributor

GitHub Issue (If applicable): closes #

PR Type

What kind of change does this PR introduce?

What is the current behavior?

What is the new behavior?

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tested code with current supported SDKs
  • Docs have been added/updated which fit documentation template. (for bug fixes / features)
  • Unit Tests and/or UI Tests for the changes have been added (for bug fixes / features) (if applicable)
  • Wasm UI Tests are not showing unexpected any differences. Validate PR Screenshots Compare Test Run results.
  • Contains NO breaking changes
  • Updated the Release Notes
  • Associated with an issue (GitHub or internal)

Other information

Internal Issue (If applicable):

Copy link
Contributor

Copilot AI left a 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 walkthrough documentation for Uno Extensions features, creating practical guides for developers integrating various capabilities into their apps.

  • Adds 18 new markdown documentation files covering authentication, configuration, HTTP clients, logging, and more
  • Provides step-by-step instructions with code examples for each feature
  • Establishes a consistent documentation structure across all walkthroughs

Reviewed Changes

Copilot reviewed 20 out of 20 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
WritableConfiguration.md Documents how to persist user preferences via IWritableOptions
WebAuthentication.md Guides developers through web-based authentication flows
UseThemeService.md Explains runtime theme switching and persistence
Serialization.md Shows JSON serialization with source generators
RequiredEntitlements.md Details iOS/Mac Catalyst keychain configuration
Refit.md Demonstrates REST API client generation with Refit
OidcAuthentication.md Covers OpenID Connect provider integration
MsalAuthentication.md Explains Microsoft Identity platform authentication
Logging.md Introduces structured logging setup
Localization.md Documents string localization and culture switching
Kiota.md Shows OpenAPI-based client generation
InternalLogging.md Covers Uno internal logging configuration
HttpEndpointOptions.md Explains custom endpoint options for HTTP clients
Http.md Documents centralized HTTP endpoint configuration
HostingSetup.md Shows hosting and dependency injection setup
DependencyInjectionSetup.md Guides service registration and injection
Cookies.md Explains cookie-based token persistence
Configuration.md Documents configuration loading from JSON
CommunityToolkit.md Shows IoC bridge with CommunityToolkit
Authentication.md Comprehensive authentication guide with custom logic

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@unodevops
Copy link
Contributor

⚠️⚠️ The build 180228 has failed on uno.extensions.

@vatsashah45 vatsashah45 marked this pull request as ready for review October 30, 2025 14:41
@DevTKSS
Copy link
Contributor

DevTKSS commented Oct 30, 2025

@vatsashah45 looks nice! Just Idea: maybe some of those links could enhance this guide? collected some in this PR:

@kazo0 kazo0 force-pushed the dev/vs/extensions-walkthrough branch from 4946073 to 4663984 Compare October 31, 2025 01:08
@unodevops
Copy link
Contributor

⚠️⚠️ The build 180412 has failed on uno.extensions.

@unodevops
Copy link
Contributor

⚠️⚠️ The build 180517 has failed on uno.extensions.

@kazo0
Copy link
Contributor

kazo0 commented Oct 31, 2025

@copilot can you fix the markdown violations and cspell errors for this PR that are failing the CI checks?

Copy link
Contributor

Copilot AI commented Oct 31, 2025

@kazo0 I've opened a new pull request, #2930, to work on those changes. Once the pull request is ready, I'll request review from you.

@CLAassistant
Copy link

CLAassistant commented Oct 31, 2025

CLA assistant check
All committers have signed the CLA.

@kazo0 kazo0 force-pushed the dev/vs/extensions-walkthrough branch from 88feda0 to b21506c Compare October 31, 2025 21:48
@kazo0 kazo0 merged commit 7053aa3 into main Oct 31, 2025
9 of 10 checks passed
@kazo0 kazo0 deleted the dev/vs/extensions-walkthrough branch October 31, 2025 22:58
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.

7 participants