Skip to content
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

add collection directory location #2517

Closed
wants to merge 1 commit into from
Closed

add collection directory location #2517

wants to merge 1 commit into from

Conversation

budparr
Copy link

@budparr budparr commented Jun 14, 2014

Currently, a Collection's "relative_directory" is set to the Collection label. The documentation is a bit confusing here, because one can apparently access the relative_directory, but not configure it.

This pull request gives users the ability to specify a directory location for their Collection. Rather than have the directory be limited to a collections label, I believe it will be better for users, including content creators, to optionally organize their collections into a folder or folders.

I'm adding this at an early state to open discussion to make sure I'm on the right track (code-wise and idea-wise). Maybe I'm the only person who obsesses over clean directory structures and maybe there's more comprehensive approach than what I've taken here.

Use case

src/
- _documents
     _authors  
     _books   
     _translators  
     _editors  
- _data  
- _includes  
- _layouts  

Still being inexperienced with Ruby, I've listed out what I think I need to do to accomplish this if we determine it's the best coarse of action. Comments appreciated.

  • decide on proper term (currently, I have "collection_location")
  • define configuration variable
  • set it so the location is optional
  • set the default to source
  • make the location available to Liquid
  • add tests
  • mention in documentation (right now there's nothing set for configuring collections)

@parkr
Copy link
Member

parkr commented Sep 7, 2014

Rethinking this...

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants