-
Notifications
You must be signed in to change notification settings - Fork 179
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
added to catalog layout for consistent levels #1009
Conversation
best-practices.md
Outdated
@@ -274,6 +275,9 @@ ended up doing. Following these recommendations makes for more legible catalogs. | |||
5. Items should be stored in subdirectories of their parent catalog. | |||
This means that each item and its assets are contained in a unique subdirectory. | |||
6. Limit the number of items in a catalog or sub-catalog, grouping / partitioning as relevant to the dataset. | |||
7. In a nested hierarchy of catalogs and collections make the different levels 'homogeneous' - all catalogs or all collections. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That sounds a bit like there should be only one level of collections, although it should also be fine to have e.g. all levels being collections (although an Item can only link to one collection).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, I'll try again. I was thinking about that as I was writing it but didn't manage to find a way to succinctly get it across. This was weirdly more challenging to explain than I was expecting.
Ok, new attempt is up. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggested some wording changes to make it a bit more clear.
Co-authored-by: Rob Emanuele <[email protected]>
Related Issue(s): #949
Proposed Changes:
PR Checklist: