Skip to content

fix: clarify request URL publish guidance - #9563

Open
Mia Miao (miachillgood) wants to merge 2 commits into
Azure:mainfrom
miachillgood:codex/issue-8559-publish-request-url
Open

fix: clarify request URL publish guidance#9563
Mia Miao (miachillgood) wants to merge 2 commits into
Azure:mainfrom
miachillgood:codex/issue-8559-publish-request-url

Conversation

@miachillgood

@miachillgood Mia Miao (miachillgood) commented Aug 23, 2026

Copy link
Copy Markdown

Commit Type

  • feature: A new feature
  • fix: A bug fix
  • docs: Documentation only changes
  • style: Changes that do not affect the meaning of the code
  • refactor: A code change that neither fixes a bug nor adds a feature
  • perf: A code change that improves performance
  • test: Adding missing tests or correcting existing tests
  • build: Changes that affect the build system or external dependencies
  • ci: Changes to CI configuration files and scripts
  • chore: Other changes that don't modify src or test files
  • revert: Reverts a previous commit

Risk Level

  • Low: Minor changes with minimal impact
  • Medium: Changes that may affect existing functionality
  • High: Significant changes that could cause breaking changes

What & Why

Closes #8559.

Request and A2A Request URLs are generated after a workflow is published, so the current "after save" descriptions give users the wrong lifecycle step. All four affected descriptions now tell users to publish the workflow to generate the URL.

Impact of Change

  • Users: See accurate guidance for when the generated URL becomes available.
  • Developers: No API or behavioral changes.
  • System: Copy-only manifest changes.

Test Plan

  • Unit tests added/updated
  • E2E tests added/updated
  • Manual testing completed
  • Tested in:
    • pnpm --filter @microsoft/logic-apps-shared build:lib
    • pnpm --filter @microsoft/logic-apps-shared test:lib (52 files, 966 tests)

Contributors

Mia Miao (@miachillgood)

Screenshots/Videos

Not applicable; this is a text-only correction.

Copilot AI balanced review requested due to automatic review settings August 23, 2026 06:57
@github-actions

Copy link
Copy Markdown
Contributor

🔒 AI Validation Pending

This PR is from an external contributor. AI validation will be performed after manual review.

Maintainers: Add the external-approved label to enable AI validation.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates Request and A2A Request manifest guidance to reflect publish-based URL generation.

Changes:

  • Replaces “after save” with publish terminology across four descriptions.
  • Covers base and Consumption A2A manifests.

Reviewed changes

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

File Description
base/manifests/request.ts Updates HTTP URL and method guidance.
base/manifests/a2arequest.ts Updates Agent URL guidance.
consumption/manifests/a2arequest.ts Updates Consumption Agent URL guidance.
Suppressed comments (1)

libs/logic-apps-shared/src/designer-client-services/lib/base/manifests/request.ts:30

  • “After publish” is grammatically incomplete because publish is a verb, unlike the noun save it replaces. Use a user-directed sentence so this customer-facing guidance remains clear.
          description: 'URL will be generated after publish',

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

type: 'string',
title: 'Agent URL',
description: 'URL will be generated after save',
description: 'URL will be generated after publish',

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Addressed in f2216de. The copy now uses the user-directed sentence: Publish the workflow to generate the URL.

type: 'string',
title: 'HTTP URL',
description: 'URL will be generated after save',
description: 'URL will be generated after publish',

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Addressed in f2216de. The copy now uses the user-directed sentence: Publish the workflow to generate the URL.

type: 'string',
title: 'Agent URL',
description: 'URL will be generated after save',
description: 'URL will be generated after publish',

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Addressed in f2216de. The copy now uses the user-directed sentence: Publish the workflow to generate the URL.

@miachillgood Mia Miao (miachillgood) changed the title fix: use publish in request URL descriptions fix: clarify request URL publish guidance Aug 23, 2026
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.

[New Experiment]: Request Trigger text still references 'Save' instead of 'Publish' in new Logic Apps experience

2 participants