Discourse GitHub

RCheesley · October 2, 2019, 4:15pm

Just adding my voice, would love to see this functionality to have the same kind of behaviour with Github Issues as with commits and PR’s.

sam · October 2, 2019, 10:46pm

I am open to a PR that adds support for something here, getting security right is tricky, we have done something in discourse code review but you need a token and you don’t want it to be your token, instead one of a throwaway github account that has no access to repos you care about.

devporto · November 22, 2019, 7:55am

We are trying this and don’t believe it’s working so it’s possible the configuration is not correct?

is the plugin limited to github projects that we own and create?
can github badges be assigned to any contribution?
where should the github linkback projects and github badges repos link to? issues? staging? develop?

Can someone use Discourse github as an example? github.com/discourse/discourse

jomaxro · January 28, 2020, 8:54pm

A question that came up recently: why aren’t we populating the admin badges page with the badge SQL? This has caused some confusion thinking that the badges aren’t working due to missing SQL, when the issue was in fact mismatched emails.

Other built-in badges (from core) show the SQL.

david · January 28, 2020, 9:03pm

It’s because the badge granting uses some pretty complex ruby logic. There are a few similar core badges (e.g. First Emoji) which use ruby logic for granting. Maybe we should add some text to ‘automatic’ badges which have no SQL to avoid confusion.

(Also, note that most sites don’t have badge SQL enabled, so this confusion is limited to ‘power users’ who have enabled the setting.)

jomaxro · January 28, 2020, 9:15pm

I do think that would help. Especially as those “power users” are the ones more likely to be looking at badges in detail.

jrudolph · February 5, 2020, 9:32am

It seems the plugin currently suffers from using a deprecated github authentication mode. So, the owner of the github organization/project will get warning mails like these (since this or last week):

On your application XYZ used an access token (with the User-Agent Faraday v1.0.0) as part of a query parameter to access an endpoint through the GitHub API.

The underlying problem seems to have been fixed in octokit but maybe not yet released. Once there is a release discourse-github needs to update the dependency and release a new version.

(We are having the issue on a hosted instance, so hopefully the fix will trickle down to that soon, for now we disabled the plugin)

sam · February 5, 2020, 10:49pm

We plan to upgrade this in the next day or 2, the new gem supporting the new method of auth was just released today. cc @david

david · February 6, 2020, 11:44am

The discourse-github plugin is now updated:

We’ll be rolling these changes out to our hosting in the coming days, but if you let me know your site address (here or by PM) I can fast track it for you @jrudolph.

pgr · February 7, 2020, 2:38pm

Hey everyone. I recently ran into this

Does that look to you as if it is a problem on my system, or an actual broken dependency in this plugin that needs updating by maintainers? If so, is that an issue that you are aware of, or should I report it somewhere?

Simon_Cossar · February 7, 2020, 5:26pm

The error I see in the error logs from your other post is Gem::MissingSpecError: Could not find 'addressable' (>= 2.3.5). It is possible that this issue has been fixed by the commit that bumps Octokit to 4.16. Have you tried installing the Discourse GitHub plugin again?

pgr · February 8, 2020, 1:06pm

Nope, sorry. Same error.

Let me make sure I’m doing this as expected: I am simply running the rebuild again, hoping the git clone will get the updated plugin, is there anything else I should be doing?

pfaffman · February 11, 2020, 12:16pm

Are you on tests-passed?

pgr · February 11, 2020, 12:18pm

I’m not sure, how can I check that? It’s a Docker install of Discourse, for production.

pfaffman · February 11, 2020, 12:19pm

It’s the default, so if you set it up you’d probably remember changing it. So this probably isn’t why your upgrade isn’t working

jrudolph · February 13, 2020, 12:23pm

Thanks a lot for the quick turn around. No need for a fast track but would be good to know once it has been deployed to the hosted instances. What will be the version of the fixed plugin?

david · February 13, 2020, 1:44pm

The plugin version is 82044bf, although we don’t display that anywhere on hosted sites. If you’re on our free, standard or business tiers, then you already have this update

jrudolph · February 13, 2020, 1:59pm

Great! It shows version 0.3 on the Admin/Plugins screen, but it’s unclear how that would map to commits:

Thanks, anyway!

jtraulle · February 15, 2020, 9:28am

Hello,

Is there any way to translate the name and description of badges ?
I run a Discourse in a non English speaking community and it would be great.

I have tried to edit the existing badges but somehow it seems that the plugin find them by their names because when changing them, the plugin just recreate new badges with the English names and descriptions.