-
Notifications
You must be signed in to change notification settings - Fork 69
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
Reuse sheet functions #98
Comments
@jameesjohn |
Oops, my bad. |
Hi, I would like to work on this issue. Can someone guide me how to get started as I am still trying to understand the codebase. |
Hi, @riyasingh1004. I'm available to guide you to understand the codebase. |
@jameesjohn thank you. Should I ask all the doubts here or is there a separate channel for the project. Also can you tell me how to get started with this issue. |
We primarily use hangouts for communication. However, make sure you have signed the CLA which can be found here. Once you are done with that, you can send me a message on hangouts (my email is available on my profile). Thanks. |
Thank you, I have sent you the message request in hangout and have signed the CLA. |
@vojtechjelinek Can I take up this issue? |
@vojtechjelinek @jameesjohn if this issue is up for grabs i would like give it a try. |
Hey @jameesjohn, can I work on this issue? |
This issue is currently blocked on #195 which is why we can't assign it to anyone. |
Note: #195 is already done, so this issue should now be available again. |
I want to work on this issue . Please assign me this issue . |
The following functions in the
checkIssueAssignee
module are performing the same functionality as in theapiForSheets
module:hasSignedCla
,checkAssignees
, and need to be refactored to use the functions in theapiForSheets
module.Note that the functions in
apiForSheets
might need to be refactored support usage in both modules.The text was updated successfully, but these errors were encountered: