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

docs: correct nextjs integration error handle #176

Merged
merged 6 commits into from
Jul 8, 2024
Merged

Conversation

tjx666
Copy link
Contributor

@tjx666 tjx666 commented Jul 7, 2024

Copy link

vercel bot commented Jul 7, 2024

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

Name Status Preview Comments Updated (UTC)
react-dev-inspector ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 8, 2024 5:23am

@zthxxx
Copy link
Owner

zthxxx commented Jul 8, 2024

Ohhhh, they changed it.

@zthxxx
Copy link
Owner

zthxxx commented Jul 8, 2024

Please enable the Allow edits from maintainers option for this PR, which make me could update related docs and merge them into your branch.

Never mind, you already enabled it.

The error I got, error: Authentication error: Authentication required: You must have push access to verify locks, was caused by git-lfs/git-lfs#2291.

It’s resolved now.

@zthxxx zthxxx added the documentation Improvements or additions to documentation label Jul 8, 2024
@zthxxx zthxxx merged commit 1ab6935 into zthxxx:dev Jul 8, 2024
2 checks passed
@tjx666
Copy link
Contributor Author

tjx666 commented Jul 13, 2024

@zthxxx 请教一下你是怎么做到在我的 pr 上提交代码的?我有个仓库也收到 pr,我想在他 pr 的分支改点东西

@zthxxx
Copy link
Owner

zthxxx commented Jul 14, 2024

@tjx666 当对方开启 Allow edits from maintainers option 的时候,你就有对应分支 push 权限

配合 GitHub Cli gh pr checkout 176 就能方便的拉到本地

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork#enabling-repository-maintainer-permissions-on-existing-pull-requests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants