-
Notifications
You must be signed in to change notification settings - Fork 18
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
Update directories.md #251
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. |
|
||
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
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