Support for Jekyll's collections_dir

[alzeih]

Listing entries with Jekyll's configuration option collections_dir set results in a http 404 error.

This diff fixes the 404, but no entries are displayed.

--- a/lib/jekyll-admin/path_helper.rb
+++ b/lib/jekyll-admin/path_helper.rb
@@ -55,7 +55,7 @@ module JekyllAdmin
       sanitized_path(
         case namespace
         when "collections"
-          File.join(collection.relative_directory, params["splat"].first)
+          File.join(site.config["collections_dir"], collection.relative_directory, params["splat"].first)
         when "data"
           File.join(DataFile.data_dir, params["splat"].first)
         when "drafts"

collections_dir was added to Jekyll version 3.7.0 (https://jekyllrb.com/docs/collections/)

With:

• jekyll-admin (0.7.0)
• jekyll (3.7.2)
• node -v v9.4.0

[jekyllbot]

This issue has been automatically marked as stale because it has not been commented on for at least two months.

The resources of the Jekyll team are limited, and so we are asking for your help.

If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.

If this is a feature request, please consider whether it can be accomplished in another way. If it cannot, please elaborate on why it is core to this project and why you feel more than 80% of users would find this beneficial.

This issue will automatically be closed in two months if no further activity occurs. Thank you for all your contributions.

[matbrgz]

This will be a nice approuch, take a look at https://prismic.io/feature/custom-type-builder they realy do a good job in a CMS.

[jekyllbot]

This issue has been automatically marked as stale because it has not been commented on for at least two months.

The resources of the Jekyll team are limited, and so we are asking for your help.

If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.

If this is a feature request, please consider whether it can be accomplished in another way. If it cannot, please elaborate on why it is core to this project and why you feel more than 80% of users would find this beneficial.

This issue will automatically be closed in two months if no further activity occurs. Thank you for all your contributions.

[maxgardner]

Running into this today. Considering this is a standard feature of Jekyll, it seems like this should be marked as a bug.

[alzeih]

This should be reopened as it is still a bug with jekyll-admin (0.8.1).