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

Remove onegraph proxy for GitHub #140

Merged
merged 2 commits into from
Mar 24, 2023
Merged

Remove onegraph proxy for GitHub #140

merged 2 commits into from
Mar 24, 2023

Conversation

dwwoelfel
Copy link
Contributor

Uses the relay compiler to generate next.js API routes by abusing the "persistConfig". The generated API routes will check that the variables are allowed and only allow the whitelisted variables to be used in the query.

Also upgrades next.js and Relay

Now that we're not hitting OneGraph, it will require a GitHub token, which you can get from https://github.com/settings/tokens.

The token only needs the "public_repo" scope.

Once you have the token, add it to a new environment variable called GITHUB_TOKEN. Then you should be able to deploy.

Note that mutations (like adding comments and reactions) will no longer work

@vercel
Copy link

vercel bot commented Mar 20, 2023

Someone is attempting to deploy a commit to a Personal Account owned by @bdougie on Vercel.

@bdougie first needs to authorize it.

@vercel
Copy link

vercel bot commented Mar 24, 2023

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

Name Status Preview Comments Updated
live ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 24, 2023 at 3:56AM (UTC)

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.

None yet

2 participants