Skip to content

Conversation

@l0minous
Copy link
Contributor

Problem

Currently Start Execution only outputs the execution arn to the output

Solution

Adding a call to execution detail provider after starting an execution, launching the execution details webview ( Also works with express executions )

Verification

test8


  • Treat all work as PUBLIC. Private feature/x branches will not be squash-merged at release time.
  • Your code changes must meet the guidelines in CONTRIBUTING.md.
  • License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

@l0minous l0minous requested a review from a team as a code owner July 23, 2025 20:23
@github-actions
Copy link

  • This pull request modifies code in src/* but no tests were added/updated.
    • Confirm whether tests should be added or ensure the PR description explains why tests are not required.
  • This pull request implements a feat or fix, so it must include a changelog entry (unless the fix is for an unreleased feature). Review the changelog guidelines.
    • Note: beta or "experiment" features that have active users should announce fixes in the changelog.
    • If this is not a feature or fix, use an appropriate type from the title guidelines. For example, telemetry-only changes should use the telemetry type.

Copy link
Contributor

@anthonyting anthonyting left a comment

Choose a reason for hiding this comment

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

should we instead mimic the console experience? i.e. by default it replaces the current tab, and an option to open in a new tab?

@zelzhou
Copy link

zelzhou commented Jul 23, 2025

should we instead mimic the console experience? i.e. by default it replaces the current tab, and an option to open in a new tab?

My suggestion is to keep it as is, and once we connected the New Execution button in the Execution Details page with the Vue StartExecution view, we can switch to replace the current tab and customers can use the New Execution button to start a new execution.

Copy link

@zelzhou zelzhou left a comment

Choose a reason for hiding this comment

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

I believe you will need to include a changelog for this change.

@l0minous l0minous changed the title feat(stepfunctions) Start Execution launching Execution Details feat(stepfunctions): Start Execution launching Execution Details Jul 23, 2025
@l0minous l0minous requested review from anthonyting and zelzhou July 23, 2025 22:13
@l0minous l0minous requested a review from anthonyting July 23, 2025 23:15
@l0minous l0minous requested a review from anthonyting July 23, 2025 23:27
Copy link
Contributor

@laileni-aws laileni-aws left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks for adding demo video

@laileni-aws laileni-aws merged commit e75caf3 into aws:feature/stepfunctions-execution Jul 24, 2025
22 checks passed
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.

4 participants