Skip to content

Add redirectUrl to step options in StepData.#620

Open
bparth24 wants to merge 3 commits intomainfrom
bparth24-issue-590
Open

Add redirectUrl to step options in StepData.#620
bparth24 wants to merge 3 commits intomainfrom
bparth24-issue-590

Conversation

@bparth24
Copy link
Copy Markdown
Contributor

@bparth24 bparth24 commented Apr 7, 2026

  • Add redirectUrl property to StepData schema in oas.yml.
  • Add prose for redirectUrl as a step option in index.html.

Addresses #590.

- Add redirectUrl property to StepData schema in oas.yml.
- Add prose for redirectUrl as a step option in index.html.

Addresses #590.
@bparth24
Copy link
Copy Markdown
Contributor Author

bparth24 commented Apr 7, 2026

@dlongley

Disclaimer: In the issue there was a link provided:

I think it would go in the OAS in here: https://github.com/w3c-ccg/vcalm/blob/7d15e7054b0f4df5ba13a64ea12acf32aa00513e/oas.yaml#L980

I could not access or see the exact line referenced above. My understanding and assumption is that the redirectUrl property should be part of stepData, based on which I made the changes.

@bparth24 bparth24 self-assigned this Apr 7, 2026
@dlongley
Copy link
Copy Markdown
Contributor

dlongley commented Apr 7, 2026

@dlongley

Disclaimer: In the issue there was a link provided:

I think it would go in the OAS in here: https://github.com/w3c-ccg/vcalm/blob/7d15e7054b0f4df5ba13a64ea12acf32aa00513e/oas.yaml#L980

I could not access or see the exact line referenced above. My understanding and assumption is that the redirectUrl property should be part of stepData, based on which I made the changes.

The link has moved to:

vcalm/oas.yaml

Line 980 in 7d15e70

StepData:

(replace "w3c-ccg" with "w3c" in the URL).

Copy link
Copy Markdown
Contributor

@dlongley dlongley left a comment

Choose a reason for hiding this comment

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

LGTM. We might want to bikeshed the language a bit more here for greater clarity, but that could happen later in some other pass -- or once an example is added in another PR it might not be necessary.

Co-authored-by: Dave Longley <[email protected]>
Co-authored-by: Ted Thibodeau Jr <[email protected]>
@bparth24 bparth24 requested a review from TallTed April 7, 2026 19:59
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.

3 participants