Add Gitea integration #31

Merged
raucao merged 3 commits from feature/24-gitea into master 2019-04-24 13:25:07 +00:00
raucao commented 2019-04-24 12:29:39 +00:00 (Migrated from github.com)

Support for Gitea organization hooks, based on GitHub hooks code.

Please note: depends on a GitHub branch of kredits-contracts, for the Gitea account info to be serialized and deserialized.

closes #24

Support for Gitea organization hooks, based on GitHub hooks code. Please note: depends on a GitHub branch of `kredits-contracts`, for the Gitea account info to be serialized and deserialized. closes #24
bumi (Migrated from github.com) reviewed 2019-04-24 12:42:29 +00:00
bumi (Migrated from github.com) left a comment

whoop whoop, \o/

whoop whoop, \o/ ![](https://media.giphy.com/media/g6VWn4bujbz2w/giphy.gif)
@ -0,0 +26,4 @@
return Contributor.all().then(contributors => {
const contrib = contributors.find(c => {
return c.gitea_username === username;
});
bumi (Migrated from github.com) commented 2019-04-24 12:35:49 +00:00

there are contract methods for that: https://github.com/67P/kredits-contracts/blob/master/lib/contracts/contributor.js#L20-L38
should be something like: Contributor.findByAccount({gitea_username: username, site: 'gitea something})

there are contract methods for that: https://github.com/67P/kredits-contracts/blob/master/lib/contracts/contributor.js#L20-L38 should be something like: `Contributor.findByAccount({gitea_username: username, site: 'gitea something})`
bumi (Migrated from github.com) commented 2019-04-24 12:40:06 +00:00

let's first release a new kredits-contract version

let's first release a new kredits-contract version
raucao (Migrated from github.com) reviewed 2019-04-24 12:49:39 +00:00
@ -0,0 +26,4 @@
return Contributor.all().then(contributors => {
const contrib = contributors.find(c => {
return c.gitea_username === username;
});
raucao (Migrated from github.com) commented 2019-04-24 12:49:38 +00:00

This is copied from the GitHub integration code, so I won't refactor things for this functionality now. Can do that for all integrations after we're not losing Gitea kredits anymore.

This is copied from the GitHub integration code, so I won't refactor things for this functionality now. Can do that for all integrations after we're not losing Gitea kredits anymore.
raucao (Migrated from github.com) reviewed 2019-04-24 12:50:27 +00:00
raucao (Migrated from github.com) commented 2019-04-24 12:50:27 +00:00
PR is open over there: https://github.com/67P/kredits-contracts/pull/113
bumi (Migrated from github.com) reviewed 2019-04-24 13:01:08 +00:00
@ -0,0 +26,4 @@
return Contributor.all().then(contributors => {
const contrib = contributors.find(c => {
return c.gitea_username === username;
});
bumi (Migrated from github.com) commented 2019-04-24 13:01:08 +00:00

ok, maybe we should make an issue to not forget it. also that we don't use gitea_username here but the accouts array/hash somehow.

ok, maybe we should make an issue to not forget it. also that we don't use `gitea_username` here but the `accouts` array/hash somehow.
bumi (Migrated from github.com) approved these changes 2019-04-24 13:02:01 +00:00
bumi (Migrated from github.com) left a comment

utACK
once a new kredits-contract version is published and we use that npm version here, then let's merge it and deploy it

utACK once a new kredits-contract version is published and we use that npm version here, then let's merge it and deploy it
raucao added the
kredits-2
label 2025-01-23 21:25:15 +00:00
Sign in to join this conversation.
No description provided.