-
Notifications
You must be signed in to change notification settings - Fork 59
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
Remove AXIOS dependency #913
Comments
Hi, @pavlovcik can I work on this issue? |
None in ubiquibot are open for the community |
! action has an uncaught error |
Just noticed a bug regarding getLinkedPRs & getLinkedIssues. Working on it. @pavlovcik |
! action has an uncaught error |
/start |
Tips:
|
@0x4007 is it okay if I take up this task it has priority of urgent but has been assigned for a month now without any activity on it. I would have gone ahead and opened a PR but I noticed it was worked on in this PR #68, but was later reverted here #644. Does this mean this issue is no longer relevant? Or we just needed a better implementation to get rid of Axios properly. |
Always use authenticated Octokit for requests related to the repository. Axios is not authenticating as the bot, so it has no access and is throwing a 404.
@wannacfuture axios doesn't have authentication. This is a private repository so it makes sense its throwing a 404.
Originally posted by @pavlovcik in https://github.com/ubiquity/card-issuance/issues/4#issuecomment-1914357197
The text was updated successfully, but these errors were encountered: