-
Notifications
You must be signed in to change notification settings - Fork 1
add memories documentation to sidebar #79
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
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.
@geometrikz - where is the memories .md file?
@gregpeters5 https://github.com/Zenlytic/zenlytic-docs/blob/main/zenlytic-ui/memories.md <- here is the link to the memories.md file. |
@geometrikz - a number of things, and I can make the wording edits easily in GB. 1- The .md file needs some edits: Some re-wording, example Title just Memories, sub-title right below: "How memories work", make that just, "How they work". 2- Can you create screenshots in full screen. It is how all screenshots are consistently shown throughout docs. Just remove the padding. 3- The memories.md file has a URL reference to zoe_content_ingestion.md, but that is not live and i haven't seen a pull request for it. i can find the .md file but needs changes. @pblankley is the content_ingestion done?
|
@gregpeters5 I'll change the wording and the screenshots. The context ingestion markdown is already on live. It's here. https://github.com/Zenlytic/zenlytic-docs/blob/main/tips-and-tricks/zoe_context_ingestion.md |
I see why i didn't catch it! this is the URL - https://docs.zenlytic.com/tips-and-tricks/zoe_context_ingestion - but the page title is "How to steer Zoe's..." I've been using a naming convention for the .md to match the page title as close as possible. Ex. https://docs.zenlytic.com/tips-and-tricks/naming-conventions GB page title "Naming Conventions". |
@geometrikz where are we at with this? |
I've deprioritized since I'm writing a new version for the new memories launch Sept ~10. |
@geometrikz should we just close this pull request? |
No description provided.