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

Credits Page #280

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Credits Page #280

wants to merge 13 commits into from

Conversation

its-miroma
Copy link
Member

@its-miroma its-miroma commented Feb 12, 2025

  • CROWDIN_TOKEN
  • GITHUB_TOKEN
Name Link
😎 Deploy Preview https://deploy-preview-280--nimble-elf-d9d491.netlify.app/credits

@its-miroma its-miroma added needs-discussion Further discussion is needed framework Related to the website's framework (Vitepress) stage:expansion This should be expanded labels Feb 12, 2025
Copy link

netlify bot commented Feb 12, 2025

Deploy Preview for nimble-elf-d9d491 ready!

Name Link
🔨 Latest commit efad45d
🔍 Latest deploy log https://app.netlify.com/sites/nimble-elf-d9d491/deploys/67c0d31e5a23040008144244
😎 Deploy Preview https://deploy-preview-280--nimble-elf-d9d491.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@its-miroma its-miroma marked this pull request as ready for review February 12, 2025 21:02
@its-miroma its-miroma requested review from a team as code owners February 12, 2025 21:02
@its-miroma its-miroma added stage:verification This should be verified and removed needs-discussion Further discussion is needed stage:expansion This should be expanded labels Feb 12, 2025
@its-miroma its-miroma force-pushed the team branch 2 times, most recently from ee79215 to c4bc67b Compare February 15, 2025 18:26
Copy link
Member

@modmuss50 modmuss50 left a comment

Choose a reason for hiding this comment

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

Error handling needs fixing.

I do worry this docs site is becoming very complex now, I guess its fine.

Copy link
Member Author

Choose a reason for hiding this comment

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

once #293 drops I'll update it to use the Avatater

Copy link
Member Author

Choose a reason for hiding this comment

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

this still needs testing as I don't have a Crowdin token

Comment on lines +83 to +100
<VPTeamPageSection v-if="committers.length">
<template #title>{{ options!.committers.title }}</template>
<template #members>
<VPTeamMembers :members="committers" size="small" />
</template>
</VPTeamPageSection>
<VPTeamPageSection v-if="authors.length">
<template #title>{{ options!.authors.title }}</template>
<template #members>
<VPTeamMembers :members="authors" size="small" />
</template>
</VPTeamPageSection>
<VPTeamPageSection v-if="translators.length">
<template #title>{{ options!.translators.title }}</template>
<template #members>
<VPTeamMembers :members="translators" size="small" />
</template>
</VPTeamPageSection>
Copy link
Member Author

Choose a reason for hiding this comment

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

how do you prefer these to be sorted?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
framework Related to the website's framework (Vitepress) stage:verification This should be verified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants