Skip to content

fix: output version preview alias URL and id#385

Open
sidvishnoi wants to merge 5 commits intocloudflare:mainfrom
sidvishnoi:version-alias
Open

fix: output version preview alias URL and id#385
sidvishnoi wants to merge 5 commits intocloudflare:mainfrom
sidvishnoi:version-alias

Conversation

@sidvishnoi
Copy link

Closes #384

@therajumandapati
Copy link

@Maximo-Guk any chance you can review and approve this PR?

@sidvishnoi sidvishnoi changed the title fix: make version preview alias URL available in outputs fix: output version preview alias URL and id Sep 11, 2025
@kpyszkowski
Copy link

@sidvishnoi Thank you for addressing it. Appreciate it 🙇🏼

@ivoilic
Copy link

ivoilic commented Nov 13, 2025

@therajumandapati @Maximo-Guk this seems like an easy win with no breaking changes, would you please approve and merge? 🙏

@dario-piotrowicz
Copy link
Member

@sidvishnoi, thanks for the PR! and so sorry that it's not been reviewed in ages, I've just pushed a couple of commit to the branch, I hope you don't mind 🙏

Let's see to land this! 😄

Copy link
Member

@dario-piotrowicz dario-piotrowicz left a comment

Choose a reason for hiding this comment

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

Looks good to me! 😄

@jamestalmage
Copy link

@sidvishnoi - you should update action.yml so the new outputs are documented... I've done so here: jamestalmage/wrangler-action@b751265

Co-authored-by: James Talmage <james@talmage.io>
@sidvishnoi
Copy link
Author

@dario-piotrowicz When can we expect a release with this?

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.

Version aliased preview URL is not passed to outputs

7 participants