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.

Sign up for GitHub

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

Jump to bottom

Restructure TOC and address "integration" messaging #1648

Merged
merged 34 commits into from
Sep 23, 2024
Merged

Restructure TOC and address "integration" messaging #1648

merged 34 commits into from
Sep 23, 2024

Conversation

IEvangelist
Copy link
Member

@IEvangelist IEvangelist commented Sep 12, 2024 •
edited by github-actions bot
Loading

Summary

In this PR:

  • Reorganize the TOC and Hub page a bit.
    • Add "integrations" in Hub page cards where appropriate
    • Remove "Extensibility" node, replaced with "Create custom integrations" node
      • Renamed articles under this section, added redirects, and fix messaging
      • Move the "Create custom integrations" node under "Integrations" node.
  • App host overview updates:
    • Move hosting integration details to integrations article
    • Add links to integrations content
  • Integrations overview updates:
    • Add more formal definitions of "hosting" and "client" integrations.
    • Add content discussing the responsibility of each
    • Add a section describing their relationship
  • Fix manifest env var names in deployment manifest article.
  • Remove "auth" messaging and call it "security" instead.

Fixes #1611

Internal previews

📄 File 🔗 Preview link
docs/deployment/azure/local-provisioning.md docs/deployment/azure/local-provisioning
docs/extensibility/custom-client-integration.md docs/extensibility/custom-client-integration
docs/extensibility/custom-hosting-integration.md docs/extensibility/custom-hosting-integration
docs/extensibility/secure-communication-between-integrations.md docs/extensibility/secure-communication-between-integrations
docs/fundamentals/app-host-overview.md docs/fundamentals/app-host-overview
docs/fundamentals/integrations-overview.md docs/fundamentals/integrations-overview

@dotnet-policy-service dotnet-policy-service bot added the okr-quality Content-health KR: Concerns article defects/freshness or build warnings. label Sep 12, 2024
@dotnet-bot dotnet-bot added the fundamentals Content related to fundamentals. label Sep 12, 2024
@IEvangelist IEvangelist changed the title Initial updates to clear up and reorg integration messaging Restructure TOC and address "integration" messaging Sep 13, 2024
@IEvangelist IEvangelist marked this pull request as ready for review September 13, 2024 20:01
@davidfowl
Copy link
Member

@maddymontaquila made a diagram we should incorporate

@IEvangelist IEvangelist merged commit 4a214e9 into dotnet:main Sep 23, 2024
7 checks passed
@davidfowl davidfowl mentioned this pull request Sep 24, 2024
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DamianEdwards DamianEdwards DamianEdwards left review comments

@davidfowl davidfowl davidfowl left review comments

@sunnynagavo sunnynagavo sunnynagavo approved these changes

@maddymontaquila maddymontaquila maddymontaquila approved these changes

@tdykstra tdykstra Awaiting requested review from tdykstra

@jamesmontemagno jamesmontemagno Awaiting requested review from jamesmontemagno

Assignees
No one assigned
Labels
fundamentals Content related to fundamentals. okr-quality Content-health KR: Concerns article defects/freshness or build warnings.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update Aspire integration documentation
6 participants
@IEvangelist @davidfowl @DamianEdwards @maddymontaquila @sunnynagavo @dotnet-bot