Skip to content

[6.x] Inertia filesystems#18932

Merged
brandonkelly merged 12 commits into
6.xfrom
feature/inertia-filesystems
Jul 17, 2026
Merged

[6.x] Inertia filesystems#18932
brandonkelly merged 12 commits into
6.xfrom
feature/inertia-filesystems

Conversation

@brianjhanson

@brianjhanson brianjhanson commented May 20, 2026

Copy link
Copy Markdown
Contributor

Brings the filesystem config over to inertia. To handle the whole "dynamic form data" situation, we have two solutions in here. Modern and legacy.

For legacy settings, we do what we're currently doing with the fields page and serialize all the form data found within a certain container via the serializeFormInputsAsObject function.

For modern settings, the Edit.vue template provides a fsTypeSettings object that you can inject into a custom Vue component registerred for your settings.

I think we can make the modern process more elegant, but this works for the moment

@brianjhanson brianjhanson changed the title Feature/inertia filesystems [6.x] Inertia filesystems May 20, 2026
Comment thread resources/js/pages/SettingsFilesystemsEditPage.vue Outdated
Comment thread resources/js/pages/SettingsFilesystemsEditPage.vue Outdated
Comment thread resources/js/components/Filesystems/FilesystemSettings.vue Outdated
Base automatically changed from feature/inertia-ui to 6.x May 26, 2026 20:46
@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown

📚 Storybook previews

@craftcms/cpopen Storybook

No changed components detected in this Storybook.

resources/jsopen Storybook

No changed components detected in this Storybook.

@brianjhanson
brianjhanson marked this pull request as ready for review July 16, 2026 21:27
@brandonkelly
brandonkelly merged commit 81f69e2 into 6.x Jul 17, 2026
25 checks passed
@brandonkelly
brandonkelly deleted the feature/inertia-filesystems branch July 17, 2026 18: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