dwyl / dwylbot

:robot: Automating our GitHub Workflow to improve team communication/collaboration and reduce tedious repetition!
28 stars 7 forks source link

Unable to add access to dwyl/time-apple-watch-app repo #131

Closed sohilpandya closed 7 years ago

sohilpandya commented 7 years ago

I tried to follow the installation instructions and failed at the first hurdle. 😢

tried to click dwyl organisation so that I could select time-apple-watch-repo

screen shot 2017-06-22 at 18 19 09

get this error message when clicking through

screen shot 2017-06-22 at 18 19 15

I'll just assign the bug label here, but I feel this is a P1 issue as I am unable to add dwybot to a repo.

iteles commented 7 years ago

Aaaaaaaah, this might be a permissions issue 🙄

Try again now @sohilpandya

sohilpandya commented 7 years ago

@iteles still getting the same issue. 😟

SimonLab commented 7 years ago

Yes it's a permission issue. I think you have to be an admin of dwyl to install DwylBot in this case.

iteles commented 7 years ago

@SimonLab An admin of the org, not just and admin of the repo?

SimonLab commented 7 years ago

Yes I think you need to b an admin of the org. I've try to install the dwylbot on dwylbot and had the same error. A Github App is linked to an organisation and from there during the installation process you can choose to install the application on all the repositories of the organisation or just selecting one repository. It's true it's a bit strange that when you try to install an application without the right permission Github doesn't show a nice error message, but I guess Github App is still quiet new still.

iteles commented 7 years ago

In that case, we need instructions for how org admins provide access for dwylbot to new repos. I'm happy to do this.

I'd also like to test what happens if I give dwylbot access to all the repos, does dwylbot magically start commenting?

ghost commented 7 years ago

I'd also like to test what happens if I give dwylbot access to all the repos, does dwylbot magically start commenting?

I think so - as far as I understand only one admin needs to install it one time and then it works for everyone

SimonLab commented 7 years ago

To add dwylbot on new repo, the user will just need to get back to the Github configuration page of the application and find the repo on the list (ie just need to repeat the installation steps described on the Readme: https://github.com/dwyl/dwylbot#install). We can precise this on the Readme :+1:

If you decide to install dwylbot on all the repositories of the organisation, then it will be active on all of them. However dwylbot will only check for new errors, it won't be able to scan the repos for existing errors

ghost commented 7 years ago

I will add these instructions to the readme:

dwylbot.herokuapp.com Manage dwylbot installations Configure Pick Organisation Repository access Only select repositories Select a repository Save

ghost commented 7 years ago

Looks like this (https://github.com/dwyl/dwylbot/issues/131#issuecomment-321338154) has already been done

https://github.com/dwyl/dwylbot/#install