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

docs: new anchor docs build with fumadocs #3493

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

ZYJLiu
Copy link
Contributor

@ZYJLiu ZYJLiu commented Jan 15, 2025

This PR updates the Anchor documentation using fumadocs.
Improvements include new pages, reorganized sections, and additional context added for various anchor features.
A preview of the new layout here.

Copy link

vercel bot commented Jan 15, 2025

@ZYJLiu is attempting to deploy a commit to the coral-xyz Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Collaborator

@acheroncrypto acheroncrypto left a comment

Choose a reason for hiding this comment

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

This is so much better than the existing docs, amazing job!

I did not review the content changes in depth, but overall, it LGTM.

Left a few comments regarding missing routes. We can get this in afterwards.

docs/app/(home)/page.tsx Outdated Show resolved Hide resolved
docs/redirects.json Show resolved Hide resolved
docs/redirects.json Outdated Show resolved Hide resolved
docs/redirects.json Outdated Show resolved Hide resolved
docs/redirects.json Outdated Show resolved Hide resolved
docs/vercel.sh Outdated Show resolved Hide resolved
@acheroncrypto acheroncrypto added the documentation Improvements or additions to documentation label Jan 19, 2025
Copy link
Collaborator

@acheroncrypto acheroncrypto left a comment

Choose a reason for hiding this comment

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

Thanks! There is a small conflict due to the changes coming from #3469 — could you rebase this branch to fix the conflict?

@ZYJLiu
Copy link
Contributor Author

ZYJLiu commented Jan 24, 2025

Thanks! There is a small conflict due to the changes coming from #3469 — could you rebase this branch to fix the conflict?

Not super familiar with rebasing, but I think I did it

Copy link
Collaborator

@acheroncrypto acheroncrypto left a comment

Choose a reason for hiding this comment

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

Not super familiar with rebasing, but I think I did it

Conflict resolved, so looking good. Thanks!

docs/app/layout.tsx Outdated Show resolved Hide resolved
docs/next.config.mjs Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants