-
Notifications
You must be signed in to change notification settings - Fork 1.7k
docs: quickstart and homepage improvements #2008
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
Conversation
Sync dev branch to docs branch
Update the docs by adding examples of the latest documentation by Pena
Update the configuration file which shows the Quickstart menu in the Navbar
@hiento09 for when you're back. Is there a way to enable autodeploy to github pages, with our ci, so we can preview the docs contributions? |
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.
Thank you - left a few change requests
Suggestion: it's probably easier to work on smaller PRs, in order to get unblocked and generally move faster
|
||
## Changes | ||
|
||
- feat: Add Jan Hacker House event page to Docs |
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 like including the changlog in our docs - it highlights how fast our codebase moves.
However, manually copying this info is redundant, since we can just point users to the github release pages.
Is there a way to automate this, or just do a simple link-out.
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 think we could implement GitHub API to fetch the data from the GitHub Releases page and programmatically input it into Jan AI docs.
|
||
::: | ||
|
||
### Stable Releases |
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.
formatting, indentation seems off?
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.
Will double check it.
docs/docs/quickstart/models-list.mdx
Outdated
sidebar_position: 3 | ||
--- | ||
|
||
# Supported Models |
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.
We support all GGUF models and more. I like that this page helps with SEO, by including a lot of model names, but I don't want users to get the wrong idea that these are the only models we support. Is there a way to convey both
docs/docs/quickstart/quickstart.mdx
Outdated
</TabItem> | ||
</Tabs> | ||
|
||
2. After you run Jan, the program will take you to the Chatting window, with list of threads and each thread is a chatting box between you and the AI model. |
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.
Threads, not "chatting"
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.
Ok I will update it.
|
||
To learn more about system requirements for your operating system, go to [Installation guide](/docs/install). | ||
|
||
### Step 2: Select AI Model |
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.
This page is helpful. Thanks for including it.
<div className={clsx(styles.cards)}> | ||
<div className={clsx(styles.card, styles.cardBluish)}> | ||
<div> | ||
<p>Lorem ipsum dolor</p> |
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.
placeholder text intentional?
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.
Yes it is intentional. Will update it once we have the full doc complete
<img alt={"Feature logo"} src={"/img/homepage-new/bg-rocket.png"}/> | ||
</div> | ||
<div className={styles.cardContent}> | ||
<h3>Praesent vel Felis Vehicula</h3> |
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.
placeholder text intentional?
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.
Yes it is intentional. Will update it once we have the full doc complete
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.
why do we need this?
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, will remove.
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.
seems like we're reincluding a lot of docusaurus default assets with this PR. why do we need these again?
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, we forgot to remove them. Will remove it
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.
Hey guys, not sure why I was re-requested to review this as there are unresolved comments.
Let's split this PR into smaller ones? Thank you!
Content updated for models, integration and extensions section
Hi @cahyosubroto @aindrajaya, as a new update, we recently merge the new structure of main page to dev branch. #1941 with preview available at https://dev.jan.ai/. Thus, some of content in this PR is currently outdated when you open PR from your folk branch to Because this PR is created from the folk repo (not have an access to Cloudflare API key deployment), when you open the PR, it won't create the preview demo. There are 2 solutions.
|
Thanks, @hieu-jan, for the prompt update and for notifying us about the recent merge of the new main page structure into the dev branch. I will thoroughly review the PR and make the necessary adjustments to ensure that the content aligns seamlessly with the new structure from the dev branch. cc: @cahyosubroto |
Describe Your Changes
In this pull request, I focused on enhancing the overall user experience and accessibility of the documentation website. Here are the key changes:
Added Quickstart Documentation:
Revamped Homepage Design:
Fixes Issues
Self Checklist
Additional Notes:
Include any additional information that might be helpful for reviewers, such as specific challenges faced, notable decisions made, or any dependencies introduced.
cc: @aindrajaya