Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[1.x] Export InertiaFormProps in Vue and Svelte Adapters #1734

Merged
merged 3 commits into from
Sep 26, 2024

Conversation

Spice-King
Copy link
Contributor

This is needed to solve type issues for a laravel/precognition PR that I'm creating. I blame Typescript for this, as for some reason, TS needs the type exported to be used as part of another packages exports, even indirectly, otherwise you can get TS error 4023. Precognition only does Vue 3, but also added it to Vue 2 for constancy reasons.

Spice-King added a commit to Spice-King/precognition that referenced this pull request Nov 22, 2023
Requires inertiajs/inertia#1734 to be published to a version.
Spice-King added a commit to Spice-King/precognition that referenced this pull request Nov 22, 2023
Requires inertiajs/inertia#1734 to be published to a new version.
Spice-King added a commit to Spice-King/precognition that referenced this pull request Nov 22, 2023
Requires inertiajs/inertia#1734 to be published to a new version.
Spice-King added a commit to Spice-King/precognition that referenced this pull request Nov 22, 2023
Requires inertiajs/inertia#1734 to be published to a new version.
@pedroborges pedroborges changed the title Export Vue's InertiaFormProps [1.x] Export InertiaFormProps in Vue and Svelte Adapters Sep 23, 2024
Copy link
Collaborator

@pedroborges pedroborges left a comment

Choose a reason for hiding this comment

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

The React adapter already exports InertiaFormProps, I added the same change to the Svelte adapter.

@pedroborges
Copy link
Collaborator

@Spice-King Could you please confirm whether we need to export InertiaFormProps from the adapter's index files, as suggested in #1596?

@pedroborges pedroborges added vue 2 Related to the vue adapter svelte Related to the svelte adapter vue 3 Related to the vue 3 adapter labels Sep 23, 2024
@pedroborges pedroborges merged commit 46a74a5 into inertiajs:master Sep 26, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
svelte Related to the svelte adapter vue 2 Related to the vue adapter vue 3 Related to the vue 3 adapter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants