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

Rework the package page #76

Closed
3 tasks done
Barbapapazes opened this issue Aug 25, 2023 · 3 comments
Closed
3 tasks done

Rework the package page #76

Barbapapazes opened this issue Aug 25, 2023 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@Barbapapazes
Copy link
Member

Barbapapazes commented Aug 25, 2023

@Barbapapazes Barbapapazes added the enhancement New feature or request label Aug 25, 2023
@Barbapapazes Barbapapazes self-assigned this Aug 25, 2023
@igorjacauna
Copy link

Hello!

Maybe add the README.md content from package in page body using the ungh?

@igorjacauna
Copy link

Hello!

I drafted a PR to collaborate. But I need help with two points:

  1. The title and description from README.md in packate repo make redundant in page. Any ideas how to slice this from README.md content to keep the title and description that already exsists? I thought of something like slice the children the first item in array from fetch in api/reame-parser.
  2. Using useAsyncData only fetch in SSR not in client side. So, when navigating in preview mode, the content is not showing, only when refresh the page.

@Barbapapazes
Copy link
Member Author

Hello!

I drafted a PR to collaborate. But I need help with two points:

  1. The title and description from README.md in packate repo make redundant in page. Any ideas how to slice this from README.md content to keep the title and description that already exsists? I thought of something like slice the children the first item in array from fetch in api/reame-parser.
  2. Using useAsyncData only fetch in SSR not in client side. So, when navigating in preview mode, the content is not showing, only when refresh the page.

Hello, thanks you so much for taking time to help! I've created another PR because I encountered a lot of issue which with the build model and with Nuxt Content (like nuxt/content#2280). To remove the title, I use the AST of the package.

In order to have the correct behavior when navigating, I completely move to a static build (using nitro: { static }).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants