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

feat(docs): add contributors & enhance AddonGallery components #449

Merged
merged 2 commits into from
Sep 20, 2024

Conversation

WRXinYue
Copy link
Collaborator

According to Xiaoyun's request, the addon author is now displayed in addons/gallery. In addition, some interactive search logic has been added to AddonGallery. More addons have been added to the documentation, and the valaxy-addon-git-log has been included to display the contributors to the documentation.

Copy link

vercel bot commented Sep 12, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
valaxy-docs-dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 19, 2024 8:18am

@WRXinYue
Copy link
Collaborator Author

Adding an index.md file under each folder in the pages directory will prevent this error. This issue arises due to the following code:

<RouterView v-slot="{ Component }">
  <component :is="Component">
    <template #default>
      <div class="prose mx-auto w-full max-w-4xl">
        <h2 :id="t('post.contributors')">
          <a :href="`#${t('post.contributors')}`" class="header-anchor" />
          {{ t('post.contributors') }}
        </h2>
        <GitLogContributor />
      </div>
    </template>
  </component>
</RouterView>

Using RouterView directly does not cause build errors, and this issue might be related to #358.

@YunYouJun
Copy link
Owner

CI is failed, some problems need to check.

@YunYouJun YunYouJun merged commit f4a3573 into YunYouJun:main Sep 20, 2024
10 checks passed
Copy link

Yun Good!

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.

2 participants