Skip to content

Conversation

@bjohansebas
Copy link
Member

Description

Note that there’s no REST API for GitHub Sponsors, only a GraphQL API. I’m still working on this.

Validation

Related Issues

closes #8199

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

@vercel
Copy link

vercel bot commented Jan 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
nodejs-org Ready Ready Preview Jan 9, 2026 5:08pm

@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2026

👋 Codeowner Review Request

The following codeowners have been identified for the changed files:

Team reviewers: @nodejs/nodejs-website

Please review the changes when you have a chance. Thank you! 🙏

@codecov

This comment was marked as off-topic.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2026

📦 Build Size Comparison

Summary

Metric Value
Old Total Size 3.74 MB
New Total Size 3.74 MB
Delta 0 B (0.00%)

Changes

➕ Added Assets (1)
Name Size
.next/static/chunks/f7b2866cd0d186f2.js 204.38 KB
➖ Removed Assets (1)
Name Size
.next/static/chunks/a1b36857c5d1da17.js 204.38 KB

method: 'POST',
headers: {
'Content-Type': 'application/json',
Authorization: `Bearer ${GITHUB_API_KEY}`,
Copy link
Member Author

Choose a reason for hiding this comment

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

@nodejs/web-infra Can I use this token, or is it better to create a new one?

Copy link
Member

@MattIPv4 MattIPv4 Jan 9, 2026

Choose a reason for hiding this comment

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

I don't believe NEXT_GITHUB_API_KEY exists. It was removed in #8163

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, okay, so can we add them then? It doesn’t require special permissions

Copy link
Member

Choose a reason for hiding this comment

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

fyi @ovflowd this should be for @openjs-vercel

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 should make this clearer what it is, perhaps we call it NEXT_GITHUB_READ_API_KEY?

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.

Implement GitHub Sponsors data fetching

4 participants