-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Update has_many show pages #2374
Conversation
make sure to use root collection template
Are you able to add a test for this? I feel like even if it's not a perfect solution, but we've got a test around the regression then it'll do for now and we can revisit it in future if necessary! |
ff12008
to
84e37c9
Compare
84e37c9
to
aab7a40
Compare
After adding rspec, I realized that the original solution did not work. I started to think about a different way to fix the issue. We might need another namespace for the collection template of has_many fields. |
a75ed29
to
7f72cfb
Compare
7f72cfb
to
0f02833
Compare
Thank you @jubilee2! I have been looking into this and now I think that is a rather complex problem 🤯 I have published a more detailed study in the issue page (see #2370 (comment)). I think the PR should close for now and discussions should happen in the issue page, until we have a better idea of how we want to tackle this. |
Thank you. |
I attempted to fix #2370, but I am not confident this it is a perfect solution.