-
Notifications
You must be signed in to change notification settings - Fork 7
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
Documentation updates #22
Conversation
Deploying with Cloudflare Pages
|
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.
Nice work! we finally have some fresh docs :D
just added some nitpicks
Co-authored-by: Maha Hajja <[email protected]>
Co-authored-by: Maha Hajja <[email protected]>
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.
Great work, Lovro!
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.
Fantastic work Lovro!
This PR updates our documentation to be more up-to-date. It is not perfect, but it is hopefully better than what was there before.
Sidenote about the external link - moving "Connector List" link inside of the "Connectors" category is a pain in the a$$, because the sidebar is currently autogenerated. To do this we need to manually generate the list (which I had a hard time doing as everything was breaking left and right). There is talk about supporting an
exclude
option which would be perfect for us as we could excludeconnectors
and manually generate just that category.