Skip to content

fixes #715 pass the ids to the config update page through context#716

Merged
stevehu merged 1 commit into
masterfrom
issue715
Jun 13, 2026
Merged

fixes #715 pass the ids to the config update page through context#716
stevehu merged 1 commit into
masterfrom
issue715

Conversation

@stevehu

@stevehu stevehu commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings June 13, 2026 19:18

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

This PR updates the config update route builder to propagate key target identifiers (environment/product/instance IDs) into the /app/config/update URL query string, so the Config Update page can initialize its target selection from context-driven navigation (fixing the missing-context behavior referenced in #715).

Changes:

  • Forward environment, productId, productVersionId, instanceId, instanceApiId, and instanceAppId from TaskResolvedContext into the generated query string.
  • Preserve the existing behavior of always setting scope, and forcing taskStep=instance-api for the manage-configuration task.

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

@stevehu
stevehu merged commit 8fe5120 into master Jun 13, 2026
1 check passed
@stevehu
stevehu deleted the issue715 branch June 13, 2026 19:24
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