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

Add support for user projects #7

Closed
wants to merge 1 commit into from

Conversation

maxdanilov
Copy link
Contributor

@maxdanilov maxdanilov commented Aug 27, 2022

Fixes #3


Detects a project's owner type (either user or organization) and adjusts the GraphQL call accordingly.

@maxdanilov maxdanilov force-pushed the support-user-owners branch from d01a1b7 to 51f5275 Compare August 27, 2022 10:56
@maxdanilov
Copy link
Contributor Author

@benbalter what do you think about this proposal?

@maxdanilov
Copy link
Contributor Author

@benbalter could you review this one, please?

@benbalter
Copy link
Collaborator

I'm currently on leave. Your proposal is great, but given the complexity, I might want to explore moving the logic to JS/TS so that we can add tests, etc. If I have the chance to review your PR, I will, if not, I look forward to reviewing it when I am back from leave. Thanks for the contribution!

@maxdanilov
Copy link
Contributor Author

maxdanilov commented Jan 30, 2023

Closing this as the action's codebase has been migrated to TypeScript.

@maxdanilov maxdanilov closed this Jan 30, 2023
@benbalter
Copy link
Collaborator

@maxdanilov the move to type script was, in part, to support this request. It should be easier now to support user-owned projects. I'd be happy to review a PR if you are willing to submit one.

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.

Does this only work with organization projects?
2 participants