Nodejs.dev: Requesting access for @nodejs-crowdin GitHub user to integrate with Crowdin

[zeke]

@nodejs-crowdin is the GitHub user we use to integrate Crowdin with the nodejs/i18n GitHub repository. We'd also like to be able to integrate Crowdin with this repo (nodejs/nodejs.dev).

Is someone on the @nodejs/website or @nodejs/website-redesign team able to grant @nodejs-crowdin admin access to this repo?

The scopes required by the Crowdin-GitHub integration are these: https://github.com/settings/tokens/new?scopes=repo,user,admin:repo_hook

See also https://github.com/nodejs/i18n/issues/248

cc @nodejs/i18n

[keywordnew]

Meeting consensus: No objections.

The credentials for this user exists in the secrets org, with limited access.

@nodejs/community-committee would appreciate an owner helping add this user @nodejs-crowdin as admin for this nodejs.dev repo

[alexandrtovmach]

@keywordnew Thanks for amazing news, BTW could you send me the link to calendar event or issue for this metings, to I can join in next times?

@zeke Could you try to integrate https://github.com/nodejs/nodejs.dev with https://crowdin.com/project/nodejs-website-dev/settings#integration. Thanks as always :+1:

[zeke]

Could you try to integrate

Sure, once the @nodejs-crowdin user has the required access.

[alexandrtovmach]

@zeke Does it still haven't?

[zeke]

Just tried again. Still only seeing nodejs/i18n on the list of available repos:

I think we're still awaiting a response here from someone on @nodejs/community-committee

[Trott]

I added the bot as Maintainer. If it needs more than that, please let me know!

[zeke]

Thanks @Trott!

Unfortunately, I'm not seeing this repo in the list when creating a new integration. I think admin access may be required for the integration:

cc @crowdin-support is that correct?

[denisqua]

Hi @Trott @zeke!

Yea, you're right! To be able to connect repo you should have admin access to that repo

[Trott]

OK, changed it! Closing again, but re-open if it's still not fixed.

[alexandrtovmach]

@zeke Could you try again?

[zeke]

Hmm. I tried again but I am still not seeing the github/nodejs.dev repo in the list.

Notes:

• Logged in to GitHub as @nodejs-crowdin
• Created a new PAT with these scopes: https://github.com/settings/tokens/new?scopes=repo,user,admin:repo_hook&description=Crowdin%20integration
• SSO'd into Crowdin with this GitHub user
• Created a new integration using the new token

This is what I see:

[denisqua]

@zeke are you sure that @nodejs-crowdin is participating in repo as Admin, not as Maintainer and have full admin access?

Seems like everything is fine with permissions

Also have idea to give some additional scopes, not sure whether it helps, but you may also add:

• Full control of organization hooks
• Full control of orgs and teams, read and write org projects

[Trott]

@zeke are you sure that @nodejs-crowdin is participating in repo as Admin, not as Maintainer and have full admin access?

The bot has been added to the repo as Admin, but the interface says it's "Pending invite" so maybe there's an email somewhere someone needs to click a link in or something like that?

[crowdin-support]

Hi @Trott Let us double-check everything with developers. Be sure, we will get back to you with the updates asap

[alexandrtovmach]

@Trott It seems like @zeke need to login as bot and accept invite, to become Admin and change the status Pending -> Accepted

[crowdin-support]

@Trott yes, it's just necessary to follow the invitation link which was sent to the email address of your Bot account. Then just follow it being logged in as Bot account on Github.

@zeke would you please give it a try and let us know how everything goes?

[zeke]

he interface says it's "Pending invite" so maybe there's an email somewhere someone needs to click a link in or something like that?

That was it! Thanks everyone for helping figure that out.

The integration is now set up!

cc @alexandrtovmach -- looks like a crowdin.yml config will need to be added to the nodejs.dev repo.

[alexandrtovmach]

@zeke Will do it in next couple of days