-
Notifications
You must be signed in to change notification settings - Fork 10
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
Help configuration parameter github #7
Comments
I'll write some documentation soon. Could you point the things you find hard to figure out? Brief guide:
|
Hello eneiluj, Personal token in my setting and linked account ok My doubts go to the configuration oauth apps in https://github.com/settings/developers What is the Homepage URL and the Authorization callback URL, please ? Thanks for your help |
Thanks for the detailed feedback!
The callback URL is given to you in Nextcloud admin settings: Once you have created your OAuth app in GitHub and set the client ID and client secret in Nextcloud admin settings, you should see a new button in Nextcloud personal settings to use OAuth to authenticate to GitHub.
As the search is working for you, you are successfully authenticated to GitHub. The widget should work. We might be facing a bug here. Could you look for related errors in the browser logs and in Nextcloud's server logs? |
To make authentication a bit clearer: As a Nextcloud admin you can choose to create a GitHub OAuth app or not. As a Nextcloud user, if the Nextcloud admin didn't create and configure a GitHub OAuth app, you can only authenticate with a "GitHub personal token" that you create in your GitHub settings. If a Nextcloud admin created and configured a GitHub OAuth app, then all Nextcloud users will see the "Get access with OAuth" (will be renamed in "Connect to GitHub"). The purpose of this button is to automatically get a token with OAuth. In the end, it's the same result, the Nextcloud user gets an authentication token. |
thank you for these clarifications |
Does the widget show "nothing to show" or does it seem like a bug? |
Maybe it's just that you were not mentioned recently. Notifications are filtered to only show you the most important ones:
Could it be that you don't have much activity on GitHub? 😁 |
@Ludovicis Let's stimulate the activity of your account 😉 . |
Hi, thanks for this nice integration! It would be great if you could mention in the app store intro what the scope of the access token needs to be (which rights need to be granted). |
@DavidMStraub The scopes are listed in connected accounts personal settings if no OAuth configuration was done and the "personal access token" field is visible. Why would you want to see them also in the readme?
@Proximus888 I'll see what I can do. I agree new releases are important events. About the projects you are watching, I think it's out of the widget's scope as it might generate a lot of content with "low" importance and avoid users to see what they want to focus on... Don't you think? Making it optional is a possibility indeed but I would like to avoid adding settings/complexity. |
Signed-off-by: Julien Veyssier <eneiluj@posteo.net>
@Proximus888 Releases are now displayed in the widget. This will be included in next release soon(ish). |
Oh really? I must have missed that somehow 🤔 But yes, that is perfectly fine then! |
@Proximus888 New releases are now displayed in the widget. It's included in v0.0.7 which is out. Let's close this swiss army knife issue 😁. |
Awesome, thanks |
Can you add the procedure to parameter your application, please ?
Thanks,
The text was updated successfully, but these errors were encountered: