Skip to content

split show template#2954

Open
goosys wants to merge 1 commit intothoughtbot:mainfrom
goosys:split-show-template-2
Open

split show template#2954
goosys wants to merge 1 commit intothoughtbot:mainfrom
goosys:split-show-template-2

Conversation

@goosys
Copy link
Contributor

@goosys goosys commented Dec 4, 2025 •
edited
Loading

@goosys goosys mentioned this pull request Dec 4, 2025
Open
@goosys
Copy link
Contributor Author

goosys commented Dec 5, 2025

Memo (just thinking out loud)

  • local_assigns feels too loose — maybe we should pass only the required locals?
  • The partial file names probably need another look.
    • We'll be doing similar work in form/index, so it might be better to align the naming.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@goosys