Closed jdtjenkins closed 1 week ago
Latest commit: 94222e731490cd8653144154505c5b3104b8c600
The changes in this PR will be included in the next version bump.
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
Changes
This PR removes the type override for the
accept
parameter in thedefineAction
Actions function.Fernando Herrera pointed out earlier that the
accept
param fordefineAction
only suggestedjson
in the intellisense instead of bothjson
andform
.Now I'm not going to pretend I understand everything going on in the typing for the
accept
param here. But I'm pretty sure the= 'json'
here is redundant.https://discord.com/channels/830184174198718474/1252972140161142816/1252996832238506056
Before:![image](https://github.com/withastro/astro/assets/7649031/0589ffac-2ce5-41fa-ada2-0eb1efb03853)
After:![image](https://github.com/withastro/astro/assets/7649031/3e454f83-9c4d-4ae6-9740-cd945396716d)
Testing
No tests are necessary here I think?
Docs
No docs are necessary as I think this is a bug fix?