search

verypossible / atomic_cms

CMS for Atomic Asset components
1 stars 3 forks source link

Nested collections #33

Closed Mrjaco12 closed 9 years ago

Mrjaco12 commented 9 years ago

Add Nested collections as per Issue #2.

bbrock25 commented 9 years ago

does this actually work?

Mrjaco12 commented 9 years ago

lol. Good call. Yes. It does. I'll have to demo it so folks know what to setup in their code.

samcdavid commented 9 years ago

Could update the readme :trollface:

bbrock25 commented 9 years ago

@Mrjaco12 could you do a quick screencast and post to youtube? I'd like to see the approach before mergin'

Mrjaco12 commented 9 years ago

I will try to do as you suggest. I thought about documenting it in the readme, but I don't know that we want to document all field options there.

Mrjaco12 commented 9 years ago

https://www.youtube.com/watch?v=xB1ltflQANo

bbrock25 commented 9 years ago

Thanks for the video @Mrjaco12, I look forward to using it. :+1:

squaresurf commented 9 years ago

I would like to defer to @bbrock25 and @samcdavid on this as they have had a lot of input.

squaresurf commented 9 years ago

@Mrjaco12 let me know if you would like me to reveiw anyways :smile:

bbrock25 commented 9 years ago

I'm all in, LGTM

Mrjaco12 commented 9 years ago

If you have any thoughts @squaresurf, I would love to hear them. Or if Sam wants to go over it again, that's fine. Just looking for that last :+1: any way I can get it.

samcdavid commented 9 years ago

:+1: