-
Notifications
You must be signed in to change notification settings - Fork 20
Update directories.md #251
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
base: main
Are you sure you want to change the base?
Conversation
content design edits
✅ Deploy Preview for inspiring-euclid-d918c8 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
| As a minimum a Directory Item must belong to a _Primary_ Channel, and this can be any Channel which accepts Directory Items of that type e.g. School. The _Primary_ Channel is required as it is the basis for the inferred URL and breadcrumb (as described above). Items may also belong to multiple _Other_ Channel. These additional Channels do not influence the URL or breadcrumb, but will include the Item in its list, and do still impact the available Facets (as described below). | ||
|
|
||
| In order to allow users to filter Directory Items by Directory Facets, Items must be 'tagged' with the appropriate Facets. When creating\editing an Item, the available Facets is determined by the Channels it belongs to. For example, consider the following Channels: | ||
| In order to allow users to filter Directory Items by Directory Facets, Items must be 'tagged' with the appropriate Facets. When creating or editing an Item, the available Facets are determined by the Channels they belong to. For example, consider the following Channels: |
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.
I this this "they belong to" is incorrect here. We should update that to "the node belongs to". It's the channels that the current node is attached to that defines what facets are available (I think).
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.
Sorry I took so long getting back to this, I had no idea there was a response...
Anyhow, I don't think content designers know what nodes are, there is no mention of nodes on the page to help, and I'm afraid that I don't understand the suggested update. @markconroy
markconroy
left a comment
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.
Thanks for this @JonathanDHolden. Just one tiny change requested by me and I think we're good then.
content design edits