Skip to content

[Inertia 3.x] Refactoring: shared_props, hooks, etc#340

Open
skryukov wants to merge 3 commits intov3/ssr-in-developmentfrom
v3/refactorings-i-always-dreamed-of
Open

[Inertia 3.x] Refactoring: shared_props, hooks, etc#340
skryukov wants to merge 3 commits intov3/ssr-in-developmentfrom
v3/refactorings-i-always-dreamed-of

Conversation

@skryukov
Copy link
Collaborator

@skryukov skryukov commented Mar 13, 2026

Refactorings including:

  • huge simplification of inertia_share
  • move Rails extensions into initializers (fixes bundler ordering issues, pest practices)
  • merge two inertia_rails.rb files in one - always been kinda confusing to me


def conditionally_share_a_prop
self.class.inertia_share incorrectly_conditionally_shared_prop: 1
inertia_share incorrectly_conditionally_shared_prop: 1
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

it's now possible! before it would raise an error, but now it's actually safe to call it

@skryukov skryukov force-pushed the v3/ssr-in-development branch from dcf30f7 to e59cfb9 Compare March 14, 2026 13:19
@skryukov skryukov force-pushed the v3/refactorings-i-always-dreamed-of branch from 03fbfb1 to 0b38491 Compare March 14, 2026 13:26
@skryukov skryukov changed the title Refactoring: shared_props, hooks, etc [Inertia 3.x] Refactoring: shared_props, hooks, etc Mar 14, 2026
@skryukov skryukov requested a review from bknoles March 14, 2026 13:29
@skryukov skryukov marked this pull request as ready for review March 14, 2026 13:30
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.

1 participant