close
Skip to content

Updated Views::IndexGenerator to copy all partials#2452

Merged
pablobm merged 1 commit into
thoughtbot:mainfrom
goosys:fix-views-index-generator
Nov 16, 2024
Merged

Updated Views::IndexGenerator to copy all partials#2452
pablobm merged 1 commit into
thoughtbot:mainfrom
goosys:fix-views-index-generator

Conversation

@goosys
Copy link
Copy Markdown
Contributor

@goosys goosys commented Nov 1, 2023

I believe this is an oversight from when the templates were split in a previous version.
Please review.

@nickcharlton
Copy link
Copy Markdown
Member

I'm not sure if we do actually want to do this — are you finding you do override stuff like pagination, or the headers? The code here hasn't changed much in years.

One other option would be to add a new generator for overriding the specifics, or we could also just do nothing and let users figure it out!

@goosys
Copy link
Copy Markdown
Contributor Author

goosys commented Dec 20, 2023

@nickcharlton
Thank you for the review.

These templates are not generated by any generator. Shouldn't we enable them to be generated by the generator as a guide for users who want to customize the templates?

One other option would be to add a new generator for overriding the specifics

I'm prepared to create a new generator and submit a pull request. In that case, what do you think would be an appropriate name for the generator?

@pablobm
Copy link
Copy Markdown
Collaborator

pablobm commented Oct 29, 2024

Initially I wasn't keen into incorporating these partials on the generator, but I'm less sure now. I think a user wanting to customise the index views will want them, so it makes sense to provide them.

@nickcharlton
Copy link
Copy Markdown
Member

Yeah, I was thinking the same, but I think it's worthwile having now.

@goosys, would you like to revisit this? I think it needs a rebase and that's probably it.

@goosys goosys force-pushed the fix-views-index-generator branch from cf188fa to 4b7b7ed Compare November 6, 2024 15:25
@goosys
Copy link
Copy Markdown
Contributor Author

goosys commented Nov 6, 2024

Thank you for checking. I rebased.

Copy link
Copy Markdown
Collaborator

@pablobm pablobm left a comment

Choose a reason for hiding this comment

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

Thank you!

@pablobm pablobm merged commit c1f78d0 into thoughtbot:main Nov 16, 2024
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