-
Notifications
You must be signed in to change notification settings - Fork 92
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
Add a troubleshooting guide #38
Comments
Hey @pandafy great idea. Did you want to contribute some documentation in the README.md? |
I would love to but I don't have any idea which permissions are needed to be selected.
|
These are the project board scenarios I know. I need to figure out the exact different permissions for each. I have put down some assumptions for permissions. repo Full control of private repositories
admin:org Full control of orgs and teams, read and write org projects
? Not sure on this one
Where to add secrets? organisation or repository. |
I didn't need to give full control of private repositories for a public repository I used this with. We should encourage users to follow concept of |
Definitely agree on least permissions. I think for next steps we need to do a manual test with each scenario and minimal permissions to make sure they are correct. I'm happy to do it but I have some other priorities this week. |
Another state to document was found by @canarro #39 (comment) |
I'll will try to do manual testing of required permissions over the weekend. 😄 |
There is one more thing which should be added in the documentation, |
It will be good to document this too: actions/toolkit#465 (comment).
|
I feel like I should handle an error if the secret is an empty string? What do you think @rodrigondec ? |
I think it is the best case. And add to the docs about this limitation (of the github itself) with references. @alex-page will you reopen the #39 and make this modification? @conarro just commented on the issue about hist problem. Maybe it's the same? I can help with the docs 😁 |
Yeah i will re-open #39 and we can resolve it with this guide. That would be great @rodrigondec. @pandafy if you have anything you can put up in a draft PR that would be awesome. Otherwise happy to start a PR we can collaborate on together. |
Apologies @alex-page , I have been busy lately. Please go ahead to create a PR. I will try to give my inputs along the way. 😅 |
Add which permissions are needed to be selected while creating personal access token.
The text was updated successfully, but these errors were encountered: