Skip to content
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

chore: separate API docs from main site #1194

Merged
merged 5 commits into from
Sep 10, 2024
Merged

chore: separate API docs from main site #1194

merged 5 commits into from
Sep 10, 2024

Conversation

mujahidkay
Copy link
Member

Closes: #1187

This PR aims to remove agoric-sdk and endo reference docs from docs.agoric.com, adds sidebar corresponding sidebar entries and redirects them to the relevant pages.

Copy link

cloudflare-workers-and-pages bot commented Sep 9, 2024

Deploying documentation with  Cloudflare Pages  Cloudflare Pages

Latest commit: 39415a7
Status: ✅  Deploy successful!
Preview URL: https://883b2192.documentation-7tp.pages.dev
Branch Preview URL: https://separate-api-docs.documentation-7tp.pages.dev

View logs

Copy link

github-actions bot commented Sep 9, 2024

Cloudflare deployment logs are available here

@mujahidkay mujahidkay self-assigned this Sep 10, 2024
@mujahidkay mujahidkay changed the title chore: separate API docs from main site (WIP) chore: separate API docs from main site Sep 10, 2024
@@ -38,6 +38,14 @@ export const nav = [
ariaLabel: 'Agoric CLI',
link: '/guides/agoric-cli/',
},
{
text: 'Agoric API Reference',
Copy link
Member Author

@mujahidkay mujahidkay Sep 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This entry seems apt to me but open to suggestions if we wanna call it something else

@mujahidkay mujahidkay marked this pull request as ready for review September 10, 2024 08:33
@mujahidkay mujahidkay requested a review from dckc September 10, 2024 08:34
@@ -38,6 +38,14 @@ export const nav = [
ariaLabel: 'Agoric CLI',
link: '/guides/agoric-cli/',
},
{
text: 'Agoric API Reference',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we usually include SDK in the name for this stuff:

Suggested change
text: 'Agoric API Reference',
text: 'Agoric SDK API Reference',

Copy link
Member

@dckc dckc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one suggestion

@@ -229,6 +229,14 @@ export default defineConfig({
},
],
},
{
text: 'Agoric API Reference',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
text: 'Agoric API Reference',
text: 'Agoric SDK API Reference',

@dckc
Copy link
Member

dckc commented Sep 10, 2024

I presume it's OK for me to go ahead and merge this. I keep tripping over the lack of it.

@dckc dckc merged commit 687cb5e into main Sep 10, 2024
4 of 5 checks passed
@dckc dckc deleted the separate-api-docs branch September 10, 2024 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

separate generated api docs from docs.agoric.com for now
2 participants