ubiquity / ubiquibot

Putting the 'A' in 'DAO'
https://github.com/marketplace/ubiquibot
MIT License
16 stars 59 forks source link

Remove AXIOS dependency #913

Open 0x4007 opened 5 months ago

0x4007 commented 5 months ago

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.

! action has an uncaught error

@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

kamaalsultan commented 5 months ago

Hi, @pavlovcik can I work on this issue?

0x4007 commented 5 months ago

None in ubiquibot are open for the community

ubiquibot[bot] commented 5 months ago
! action has an uncaught error
wannacfuture commented 5 months ago

Just noticed a bug regarding getLinkedPRs & getLinkedIssues. Working on it. @pavlovcik

ubiquibot[bot] commented 5 months ago
! action has an uncaught error
Sadaf-A commented 3 months ago

/start

ubiquibot[bot] commented 3 months ago

Warning! This task was created over 58 days ago. Please confirm that this issue specification is accurate before starting.
DeadlineWed, Mar 27, 5:38 PM UTC
Registered Wallet 0x0BEd00438D57d07E3667b85Fa8EB86Af147C7025
Tips:
jordan-ae commented 2 months ago

@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.