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

Does this only work with organization projects? #3

Open
cp2004 opened this issue Aug 11, 2022 · 4 comments
Open

Does this only work with organization projects? #3

cp2004 opened this issue Aug 11, 2022 · 4 comments

Comments

@cp2004
Copy link

cp2004 commented Aug 11, 2022

I noticed in the readme that you have a parameter organization. I wanted to use this for a personal project board I had, which is not under any org. When unspecified, it defaults to my username (cp2004) however the action seems to fail.

The error also seems to indicate organization-only:

gh: Could not resolve to an Organization with the login of 'cp2004'.

https://github.com/cp2004/github_management/runs/7776329997?check_suite_focus=true#step:4:65

Is that the case or have I broken it? If that is the case, would you consider making it work with both or putting it in the readme?

This add to project action allows to specify the URL, rather than separating the org/project ids, and it works OK.

@benbalter
Copy link
Collaborator

The easiest way would be to fork the action and modify references to organization in the GraphQL query to user, and it should just work.

The better solution would be to abstract out the query to be able to accept a user or an organization, but I don't have the ability to do that at this time myself.

@cp2004
Copy link
Author

cp2004 commented Aug 12, 2022

The easiest way would be to fork the action and modify references to organization in the GraphQL query to user, and it should just work.

I tried this, it does seem to work. I need to make the rest of the actions work now to check everything works, but based on some testing I think it should be fine. No worries about not being able to make the changes, I hadn't realised initially how easy it might be to fork.

@maxdanilov
Copy link
Contributor

Proposing a fix in #7

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 a pull request may close this issue.

4 participants
@benbalter @maxdanilov @cp2004 and others