Skip to content

Conversation

@mauriciabad
Copy link
Contributor

Description

Documentation for the new ActionInputSchema utility type I proposed in withastro/astro#14698

I put the latest version +1 patch in the <Since> component, but I don't know the exact version yet.

Feel free to change the text if is not clear :)

I also native Spanish, if you want me to translate it.

@netlify
Copy link

netlify bot commented Oct 31, 2025

Deploy Preview for astro-docs-2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 7406c62
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/691e22765b44900008029635
😎 Deploy Preview https://deploy-preview-12647--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@astrobot-houston
Copy link
Contributor

astrobot-houston commented Oct 31, 2025

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
en/reference/modules/astro-actions.mdx Source changed, localizations will be marked as outdated.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@sarah11918 sarah11918 added add new content Document something that is not in docs. May require testing, confirmation, or affect other pages. merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!) labels Nov 3, 2025
@sarah11918 sarah11918 added the minor-release For the next minor release; in the milestone, "merge queue" when approved by Sarah! label Nov 12, 2025
@sarah11918 sarah11918 added this to the 5.16.0 milestone Nov 12, 2025
@sarah11918
Copy link
Member

Thanks for submitting this documentation with your feature @mauriciabad ! I will also ask @florian-lefebvre to check in on these docs after everything is resolved in the Astro feature PR to make sure what's here matches the decisions made over there!

Copy link
Member

@florian-lefebvre florian-lefebvre left a comment

Choose a reason for hiding this comment

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

Confirming we keep the name so docs can be reviewed!

mauriciabad and others added 3 commits November 17, 2025 09:01
Co-authored-by: Florian Lefebvre <contact@florian-lefebvre.dev>
Copy link
Member

@sarah11918 sarah11918 left a comment

Choose a reason for hiding this comment

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

Thanks for submitting docs to go with your feature, @mauriciabad ! I'll probably also need @ArmandPhilippot to help me with wording here because this is not my area of expertise and I do think we need to adjust the text here to match the Astro docs style.

I've left some comments/questions!

mauriciabad and others added 5 commits November 18, 2025 10:45
Co-authored-by: Sarah Rainsberger <5098874+sarah11918@users.noreply.github.com>
Co-authored-by: Sarah Rainsberger <5098874+sarah11918@users.noreply.github.com>
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
@mauriciabad
Copy link
Contributor Author

Thanks everyone for the reviews, I added some comments and suggestions. Make any edits you want, for me all outcomes are okay.

mauriciabad and others added 3 commits November 18, 2025 21:51
Co-authored-by: Sarah Rainsberger <5098874+sarah11918@users.noreply.github.com>
@sarah11918
Copy link
Member

sarah11918 commented Nov 18, 2025

Should this also be added to the beginning of the types reference as something new to import from astro:actions?

Since these are all in alphabetical order, let's also move the entry up above ActionReturnType and SafeResult so that it's 3rd last.

image

@mauriciabad
Copy link
Contributor Author

@sarah11918 feel free to directly commit the changes you want to make

Copy link
Member

@sarah11918 sarah11918 left a comment

Choose a reason for hiding this comment

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

Alright, this one's looking pretty good! Thanks everyone for their help with this one. Should be good to go for 5.16 this week!

@sarah11918
Copy link
Member

@yanthomasdev At your leisure.... 😄

Copy link
Member

@yanthomasdev yanthomasdev 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!

@sarah11918 sarah11918 added the Merge Queue Approved and ready to be merged (wait for feature release if also labelled M-O-R)! label Nov 18, 2025
@sarah11918 sarah11918 changed the base branch from main to 5.16.0 November 19, 2025 20:03
@sarah11918 sarah11918 merged commit c0bc150 into withastro:5.16.0 Nov 19, 2025
6 checks passed
@mauriciabad mauriciabad deleted the patch-1 branch November 26, 2025 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

add new content Document something that is not in docs. May require testing, confirmation, or affect other pages. Merge Queue Approved and ready to be merged (wait for feature release if also labelled M-O-R)! merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!) minor-release For the next minor release; in the milestone, "merge queue" when approved by Sarah!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants